Skip to content

Conversation

@christolis
Copy link
Member

Description

Use RAG with ChatGPT to store all XKCD comics and display the correct one based on the chat history.

There is also the possibility to specify your own XKCD if you wish.

Example

image

Use RAG with ChatGPT to store all XKCD comics and display the correct one
based on the chat history.

There is also the possibility to specify your own XKCD if you wish.

Signed-off-by: Chris Sdogkos <work@chris-sdogkos.com>
@christolis christolis added enhancement New feature or request new command Add a new command or group of commands to the bot java Pull requests that update java code labels Jan 31, 2026
Copy link
Member

@Zabuzard Zabuzard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the chatgptservice class should not deal with xkcd stuff. refactor the code so that anything xkcd is done in the xkcd classes instead. the responsibilities should be correct.

why does it need to do file io? i would like to avoid that if possible.

@christolis
Copy link
Member Author

the chatgptservice class should not deal with xkcd stuff

@Zabuzard I agree with you on that. The createOrGetXkcdVectorStore method you saw introduced should really have been named createOrGetVectorStore as it generally does what it describes for any situation, it's just that the XKCD feature happens to be using it exclusively. Remember that as of writing this, it's still a draft pull request for a good reason. The code does what it's supposed to do, but I am not 100% satisfied with how it looks like, and further refactors are on the way. 👍

why does it need to do file io? i would like to avoid that if possible.

Could you clarify what you mean by that? Are you trying to avoid storing files completely? Or just using file.io in particular?

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

Labels

enhancement New feature or request java Pull requests that update java code new command Add a new command or group of commands to the bot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants