-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Add "regenerate" button to AI chat tab #12191
Comments
Hello maintainers, I am interested in taking on this issue if possible! |
/assign @arshchawla21 |
👋 Hey @arshchawla21, thank you for your interest in this issue! 🎉 We're excited to have you on board. Start by exploring our Contributing guidelines, and don't forget to check out our workspace setup guidelines to get started smoothly. In case you encounter failing tests during development, please check our developer FAQs! Having any questions or issues? Feel free to ask here on GitHub. Need help setting up your local workspace? Join the conversation on JabRef's Gitter chat. And don't hesitate to open a (draft) pull request early on to show the direction it is heading towards. This way, you will receive valuable feedback. Happy coding! 🚀 ⏳ Please note, you will be automatically unassigned if the issue isn't closed within 30 days (by 16 December 2024). A maintainer can also add the "📌 Pinned"" label to prevent automatic unassignment. |
Hey maintainers! |
/unassign @arshchawla21 |
👋 Hey @arshchawla21, you've been automatically unassigned from this issue due to inactivity. Note If you'd like to be re-assigned, just leave another comment or ask a maintainer to assign you again. |
/assign @Princccee |
👋 Hey @Princccee, thank you for your interest in this issue! 🎉 We're excited to have you on board. Start by exploring our Contributing guidelines, and don't forget to check out our workspace setup guidelines to get started smoothly. In case you encounter failing tests during development, please check our developer FAQs! Having any questions or issues? Feel free to ask here on GitHub. Need help setting up your local workspace? Join the conversation on JabRef's Gitter chat. And don't hesitate to open a (draft) pull request early on to show the direction it is heading towards. This way, you will receive valuable feedback. Happy coding! 🚀 ⏳ Please note, you will be automatically unassigned if the issue isn't closed within 90 days (by 27 February 2025). A maintainer can also add the "📌 Pinned"" label to prevent automatic unassignment. |
We are currently using Langchain4j to manage LLMs in the JabRef 6.0 development version. I opened an issue over there, because I haven't found anything related in their documentation. See langchain4j/langchain4j#2215. I found a Q&A related to how the memory is structured: langchain4j/langchain4j#2035. Maybe it helps. Maybe also @InAnYan can give a pointer to where one could start in the code. |
Fixing this issue should be simple. You need to add a button for regeneration on the side of chat message, near the "delete" button. Because chat history and chat message componenta are decoupled, you need to add a callback for regenerate button in chat message component constructor. Then you need to implement "regenerate" method in chat history component, which you will pass to chat message constructor. This method will delete last AI message and call AI again |
langchain4j responded in the issue I raised:
|
Yes! This should be done manually. And in my code there should be a method for deleting both message from LLM chat memory and chat history (yeah, there are 2 histories 😄 ) |
JabRef 6.0--2024-11-13--5f1dd59
Windows 10 10.0 amd64
Java 23.0.1
JavaFX 23.0.1+4
Problem:
If the response of the AI in the AI Chat tab of the entry editor is inadequate, the whole conversation first has to be deleted and then started anew.
A workflow that involves multiple generations of a chat model to find a better response is hardly possible, if the conversation is long.
Solution that I would like:
Add regenerate button for AI chat.
Expected behaviour: Clicking the button will trigger a new response by the large language model.
Additional context
Current UI of the AI Chat tab in JabRef:
Maybe GPT4All can be an inspiration for the location:
The text was updated successfully, but these errors were encountered: