taipy icon indicating copy to clipboard operation
taipy copied to clipboard

Add Option to Delete Existing Conversation in Chatbot

Open Mefisto04 opened this issue 1 year ago • 6 comments

Description

Currently, the chatbot in Taipy does not provide an option to delete the existing conversation. Users may want to reset the chatbot to start a fresh topic or delete previous messages for clarity. This feature would enhance user experience by allowing them to clear the chat history and begin a new conversation at any time.

Solution Proposed

Implement a button in the chatbot interface that allows users to:

  • Delete all past interactions and refresh the conversation window.
  • This could be achieved by adding a "Reset" button in the UI, which when clicked, would clear the chatbot's state and allow the user to start fresh.

Impact of Solution

This feature will impact the chatbot's flow, as it needs to handle conversation state more effectively. Care must be taken to ensure that resetting the conversation does not interfere with session management or other ongoing tasks. Additionally, this feature may require updates to the chatbot's UI and conversation logic.

Additional Context

In its current form, users may have to manually scroll through or ignore past messages to start a new topic. This feature will streamline the user experience. Other chatbot solutions often provide such a reset option to help users manage conversations effectively.

Acceptance Criteria

  • [ ] Ensure new code is unit tested, and check code coverage is at least 90%.
  • [ ] Create related issue in taipy-doc for documentation and Release Notes.
  • [ ] Check if a new demo could be provided based on this, or if legacy demos could be benefit from it.
  • [ ] Ensure any change is well documented.

Code of Conduct

  • [X] I have checked the existing issues.
  • [X] I am willing to work on this issue (optional)

Mefisto04 avatar Oct 07 '24 12:10 Mefisto04

Can i work on this?

eleensmathew avatar Oct 07 '24 15:10 eleensmathew

please find another issue @eleensmathew, I have started the changes, @FredLL-Avaiga please assign this issue to me so that i can start working on it.

Mefisto04 avatar Oct 07 '24 15:10 Mefisto04

This issue should be discussed internally before it is open for contribution. Thank you for your issue; we'll give you an answer soon

FlorianJacta avatar Oct 07 '24 20:10 FlorianJacta

What chatbot are you referring to?

Are you referring to one of our LLM demos like: https://docs.taipy.io/en/latest/tutorials/fundamentals/4_chatbot/ Or are you referring to the chat visual element in the dev version: https://docs.taipy.io/en/develop/refmans/gui/viselements/generic/chat/

What is preventing you from creating a button that resets the variable storing the conversation messages?

AlexandreSajus avatar Oct 08 '24 10:10 AlexandreSajus

Hii @Mefisto04 , I can work for this feature so that the users can easily reset their conversation which is really good and if we refresh the conversation window, it might affect other crucial items. I suggest the Reset button is absolutely a good idea to clear the conversation. Kindly assign me to do this under hacktoberfest 2024.

SaumyaSubham avatar Oct 08 '24 17:10 SaumyaSubham

/assign

KrishChothani avatar Oct 14 '24 08:10 KrishChothani

@jrobinAV Please assign this issue to me and please do clarify whether this RAG chatbot or chat UI component.

Vaibhav91one avatar Oct 27 '24 19:10 Vaibhav91one

pls assign this work to me am looking forward to work on it

kekwlboy12469 avatar Oct 28 '24 07:10 kekwlboy12469

@jrobinAV Please assign this issue to me if possible, would like to contribute

Thank you

jyotiahluwalia avatar Oct 29 '24 23:10 jyotiahluwalia

No one answered @AlexandreSajus question. Deleting the current conversation in Taipy chat control is as simple as resetting the backend supporting tabular data to an empty array/dataframe ... I don't think this should be part of the Taipy control. I propose to close this issue (unless it refers to Talk To Taipy Demo) Thoughts @jrobinAV @AlexandreSajus @FlorianJacta @FabienLelaquais ?

FredLL-Avaiga avatar Oct 30 '24 08:10 FredLL-Avaiga

Agreed, let's close this

AlexandreSajus avatar Oct 30 '24 08:10 AlexandreSajus

I am OK to close it

jrobinAV avatar Nov 18 '24 09:11 jrobinAV