-
Notifications
You must be signed in to change notification settings - Fork 75
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
feat: add a sentimental analysis demo using Proton to the examples #792
base: develop
Are you sure you want to change the base?
Conversation
|
@jovezhong Can I proceed with this example? I would like to hear your thoughts on this |
Hi @varshith257 , thanks for contributing more demos for Timeplus Proton. Yes, you can go ahead prepare the PR with testing data, SQL, or even visualization and alerts. When the PR is ready to review, you will be rewarded once and when it's merged there will be more rewards. A few suggestions to make the demo more relevant and impressive:
Feel free to ask question in Discussion tab or report bugs/feature-request in this ticket or new tickets. Look forward to your PR |
@jovezhong How's about using the BERT model? I have used it for a recent project done in a hackathon :) |
I am not that familiar with BERT but my impression is it is quite common. Feel free to try it and let us know any progress or challenge. |
Maybe the challenge I forsee is of integration of Proton and BERT Model responses. Ahh, I will connect you in slack if I encounter any challenges |
Hi @varshith257 , just to check whether there is anything we can help on the blog or demo |
PR checklist:
proton: starts/ends
for new code in existing community code base ?Please write user-readable short description of the changes:
Part of #527