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

Feature/product review prebuilt model [resolves #221] #238

Open
wants to merge 8 commits into
base: develop
Choose a base branch
from

Conversation

irmakyucel
Copy link

  • Update previous PR for compatibility with latest developf6f054d.
  • Create product_sentiment.py, test_product_sentiment.py
  • Optimize and build the product review sentiment model
  • Change README file accordingly
  • Change product_sentiment.py for working with ICU tokenizer
  • Change test_product_sentiment.py due to the changed class labels
  • Add product review sentiment model to context.py file
  • TODO look for ways to improve this score. Report @dafajon with possible replacement for dataset.

@husnusensoy
Copy link
Contributor

@irmakyucel Please try to improve model accuracy and resend the pull request. Also add any relevant work with the dataset.

@husnusensoy husnusensoy added pending Pending Merge Requests question Further information is requested labels Apr 2, 2021
@dafajon dafajon self-assigned this Apr 19, 2021
@dafajon dafajon added the ready label Apr 19, 2021
@husnusensoy husnusensoy removed the ready label Apr 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pending Pending Merge Requests question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants