search

flynnoct / chatgpt-telegram-bot

Telegram bot implemented by OFFICIAL OpenAI ChatGPT API (gpt-3.5-turbo, released on 2023-03-01)
MIT License
181 stars 43 forks source link

Suggestion: Multi-User Support #20

Closed dtimekeeper closed 1 year ago

dtimekeeper commented 1 year ago

One of my use cases is basically running another instance for my friends to use as a telegram bot. It would be helpful to have a way to ‘whitelist’ all telegram IDs so as long as they know the link to message it, they can talk to the bot.

Related to this too, not sure how feasible it is, would be separate conversational context for each person that messages the bot, but I’m also not sure if this is even possible with the API.

flynnoct commented 1 year ago

I like this allow_all_user idea and I will implement this.

About maintain a context for each person, may I ask you why do you need this function? Why not chat with the bot privately if you want separate contexts?

dtimekeeper commented 1 year ago

Thank you!

So I may not be understanding the API and each bot session properly, but do all conversations with the bot from everyone share one context thread until it is reset? Or do they have their own context and conversation with each telegram ID it talks with.

On Sun, Mar 5, 2023 at 8:02 PM flynnoct @.***> wrote:

I like this allow_all_user idea and I will implement this.

About maintain a context for each person, may I ask you why do you need this function? Why not chat with the bot privately if you want separate contexts?

— Reply to this email directly, view it on GitHub https://github.com/flynnoct/chatgpt-telegram-bot/issues/20#issuecomment-1455405804, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABK3WGZYLB67TAUQFR24U6DW2VOTZANCNFSM6AAAAAAVQR22PQ . You are receiving this because you authored the thread.Message ID: @.***>

flynnoct commented 1 year ago

I think it might be a misunderstanding.

Currently, the bot will maintain contexts for each "conversation". Different users' private messages have different contexts, and different groups have different contexts. However, within the same group, they share the same context (currently the bot cannot distinguish which user is talking to it in the group).

BTW, the /clear command only works for current context and it will not touch any of other contexts.

dtimekeeper commented 1 year ago

Ah I understand now - then that is perfect already.

I think with the addition of allow_all_users, this will become a very compelling choice for people who want to roll their own Telegram bots for themselves AND their friends.

On Sun, Mar 5, 2023 at 8:14 PM flynnoct @.***> wrote:

I think it might be a misunderstanding.

Currently, the bot will maintain contexts for each "conversation". Different users' private messages have different contexts, and different groups have different contexts. However, within the same group, they share the same context (currently the bot cannot distinguish which user is talking to it in the group).

BTW, the /clear command only works for current context and it will not touch any of other contexts.

— Reply to this email directly, view it on GitHub https://github.com/flynnoct/chatgpt-telegram-bot/issues/20#issuecomment-1455418439, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABK3WG7LAXWV76G2M742QTDW2VQC7ANCNFSM6AAAAAAVQR22PQ . You are receiving this because you authored the thread.Message ID: @.***>

flynnoct commented 1 year ago

Implemented. Pull dev branch and have a try.

I strongly recommend you to create a new directory to test it, the dev channel code could be unstable. This feature will be released in v1.2.0.