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

Multiple channel handling #24

Open
indivisible-irl opened this issue Sep 28, 2012 · 5 comments
Open

Multiple channel handling #24

indivisible-irl opened this issue Sep 28, 2012 · 5 comments

Comments

@indivisible-irl
Copy link
Member

Allow the bot to run on multiple channels concurrently.

Hopefully won't need to get into multi-threading as I've heard multi-threading is a headache in python.

Anyway, this would be useful as there are already multiple channels people are using to chat. eg #learnjava is being used for JAdventure. And I have #ProgetherBotTest open for bots

@indivisible-irl
Copy link
Member Author

Pull #27 made the first steps towards this but we still need to mod a lot of the code where channels get passed around to allow for multiple channels at once.

So not finished but getting there.

@kzisme
Copy link
Contributor

kzisme commented Dec 3, 2013

Has this made any progress ,or where are we at now with this?

If not, what would be the initial steps to getting multiple channels working?

@projectdelphai
Copy link
Member

Nothing has been done in this direction. Here's what I'm thinking. In the settings, have each channel info be added as a different section of the settings (with a database per channel). So one section for #reddit-progether and another for #progether-bots, for example. Then you would start the program with python main <channel_keyword> as an arg. Each channel keyword is connected to a section in the settings. Then mainpy would start a instance of ircBot in a different thread for each channel. Each ircBot thread would only load the necessary channel information from the settings.txt. I think having a new database for each channel is probably necessary and important though that may be debated.

@kzisme
Copy link
Contributor

kzisme commented Dec 13, 2013

Well I like the direction this is going, but should we fix the settings.txt file in the other issue first? :)

@projectdelphai
Copy link
Member

This issue is very low priority. Especially seeing as it's only being used in one channel.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants