(Go: >> BACK << -|- >> HOME <<)

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

feat: add a sentimental analysis demo using Proton to the examples #792

Draft
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

varshith257
Copy link

PR checklist:

  • Did you run ClangFormat ?
  • Did you separate headers to a different section in existing community code base ?
  • Did you surround proton: starts/ends for new code in existing community code base ?

Please write user-readable short description of the changes:
Part of #527

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@varshith257 varshith257 marked this pull request as draft July 3, 2024 20:47
@varshith257
Copy link
Author
varshith257 commented Jul 3, 2024

@jovezhong Can I proceed with this example? I would like to hear your thoughts on this

@jovezhong
Copy link
Contributor

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:

  • considering using live data, instead of a static data such as CSV. The data can be mocked data or real data in the internet (I used to build a demo with twitter data, but Elon Mask he changed the pricing a lot). You may try some RAG tool to get data from hackernews/reddits or other sources
  • considering using a sophisticated sentiment analysis algo, not just if/else, or regex match. You may run Proton locally and build a REST API with other tools then register that service as a remote UDF, or build that logic with JavaScript UDF.

Feel free to ask question in Discussion tab or report bugs/feature-request in this ticket or new tickets. Look forward to your PR

@varshith257
Copy link
Author

@jovezhong How's about using the BERT model? I have used it for a recent project done in a hackathon :)

@jovezhong
Copy link
Contributor

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.

@varshith257
Copy link
Author

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

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.

None yet

3 participants