-
Notifications
You must be signed in to change notification settings - Fork 52
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
[FeatureRequest] Channel Content processable with chatgpt #28
Comments
Adding a GUI is currently out of the scope of this project, as this is an external add-on and not a Mattermost Plugin. I would prefer if we implemented this similarly to the way it has been done, above, just via regular chat. We could improve the prompting for this, of course. |
@yGuy I wouldn't expect a GUI for this. A prompt is perfect. But currently it's not possible in channels since context is missing. It only works for threads. I would be nice if we could use channel as a context, maybe limited to a concrete number of posts in a channel. |
OK, I guess the idea would be to use two queries: the first one detects whether the user was asking for a summarization and tries to infer the scope of the summarization. In case this "matches" we parse the scope from the answer, collect all entries in the scope and pass them to the LLM for summarization. That would work and I would be willing to help with the implementation if someone does the initial prompting work. |
You might want to try out version 2.0.0 which should be able to do some of this, now. It knows the names of the persons talking and has access to a method that will provide it with the necessary context. Let us know how this works for you! |
@yGuy i just updated to the new version but now the whole bot doesn't work anymore. Log output:
|
:-( This is due to #46 |
Oh oh. It seems like OpenAI does not like your username. This is a circumstance we weren't aware of.
chatgpt-mattermost-bot/src/botservice.ts Line 187 in c6203eb
This is the line where we obtain the username from mattermost. A possibility would be to remove every character from the username which does not follow the regex from OpenAI ^[a-zA-Z0-9_-]{1,64}$
|
Is it possible to use the display name instead of the username if exists? That would be more convenient. The problem with the special characters in the username is already fixed as i can see. |
We could be using the display name, too, of course. Although chances are even bigger that it won't work with OpenAI. No Spaces allowed, e.g., only very limited set of ASCIIs characters (see the regex). I don't understand why, but that's how the OpenAI API wants it to be. |
The last issue could be solved by fixing #55 |
A friend of mine has build a similar bot for slack.
One thing he can do with it, is to summarize the conversations of a channel in a specific time frame (see screenshot).
It would be nice if this bot could also do something like this. In my opinion an UI isn't needed. A short command to chatgpt with the params should be enough.
It could also be used for different commands that have channel content as context.
Result:
The text was updated successfully, but these errors were encountered: