-
Notifications
You must be signed in to change notification settings - Fork 2
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: Sentiment Analyzer #18
Conversation
It blows my mind how fast this PR was pushed up o_O I'll take a look and resolve anything I find, pushing directly to your branch if that's ok and then merge ASAP. |
other version was deprecated by poetry at some point
so installation can be skipped if necessary for quick testing
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Aight, thanks a lot again for this PR. There's still a few minor things I'd like to adjust but those can be done after merging IMO
For now, this is good enough already as is :D
Closes #13
Please double check the poetry related files, I've never used it before and it was a fight to get it working.
How it works:
The bot will cache sentiments of recent messages and warn moderators in a channel if it negative sentiments outweigh positive and neutral sentiments. I tested the performance checking each message on a Hetzner CPX21 server (which was also running two game servers at the time) and it seems like checking every message is feasible. You can specify how many messages are checked and how many messages are cached if the results or performance aren't satisfactory. I decided to not have it ping moderators as it might get spammy.
Performance demo: