Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Reviews Sentiments #171

Open
wants to merge 24 commits into
base: master
Choose a base branch
from

Conversation

Yash-Bhatnagar-02
Copy link

@Yash-Bhatnagar-02 Yash-Bhatnagar-02 commented Oct 21, 2024

Description

Added the files for the Reviews Sentiments

Issue Resolved

#170

Changes Made

Added a new model for sentiment analysis of reviews

Screenshots or Videos

Additional Details

Checklist

  • My code follows the current project structure
  • I have thoroughly reviewed and updated the requirements.txt file to include any new packages
  • The predict.py file includes a properly implemented model_details() function, or the notebook contains this function to print a detailed model report. The model will not be accepted without this function, as it is essential for generating the necessary model details.
  • I have added relevant tests (if necessary).
  • I have added comments in the code where needed.
  • This PR is submitted under Hacktoberfest.
  • This PR is submitted under GirlScript Summer of Code (GSSoC-Extd).

@yashasvini121 yashasvini121 self-requested a review October 27, 2024 02:41
Copy link
Owner

@yashasvini121 yashasvini121 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Include the jupyter notebook as well; otherwise, your model will get level 1 status.
  • Also rename Review_sentiments/ folder to review_sentiments/ (not important)
  • Sync your fork and add your model description in App.py (not important)

Copy link
Owner

@yashasvini121 yashasvini121 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let me know if you can do this; otherwise, I'll merge your PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants