DeSinc / SallyBot

AI Chatbot coded in Discord.net C#
MIT License
299 stars 51 forks source link

Added python version of Sally + Fixed some of your markdown violations #28

Closed vilewired closed 1 year ago

vilewired commented 1 year ago

I know.. even markdown has rules, don't ask. Anyways this is my python version, you can try it out yourself before you commit it to your repo I think it works pretty well and I am pretty happy that it is ready now. It can also do OCR already, everything should work fine, try it out and let me know if you like it :)

rwf93 commented 1 year ago

Outsiders perspective: I'd highly recommend splitting the ai functionality into a cog and also using .env files (making sure to gitignore them). I also recommend you to use python venvs and freeze your dependencies to a requirements.txt. (you can use CarlBot as an example)

This is also highly applicable for the C# implementation in the context of .env. Keeping secrets in a .cs file that is tracked by git is a recipie for disaster :).

vilewired commented 1 year ago

I'll do that, I got another repo open for it anyways, so far I actually had no problems running it without a venv since the packages that I use don't get major updates that could break them. I'll look into it tho, id love to make it even better

vilewired commented 1 year ago

So I implemented your Idea with the .env file here, it works quiet nicely, thanks for the advice. I also mentioned you and your bot in my repo :) I did not implement the cog thing (yet, I might do that too soon, depends on how bored I'll get) because I would need to rewrite my entire code to use classes and I don't really see the reason for my bot to use cogs, since I only listen to messages and then execute functions based on them, aka I don't have many commands nor listeners.

DeSinc commented 1 year ago

yeah I would just make a new repo, I thought about it and there's no way I'd put all this python code in here especially since I just updated everything on the c# side I would never want to have some outdated confusing .py code sitting around in files un-updated confusing people

but you should definitely make your own repo for this if you haven't already

vilewired commented 1 year ago

I did and I am also updating things ofc