MAVProxy icon indicating copy to clipboard operation
MAVProxy copied to clipboard

Chat: Restoring old chats in chat module

Open adityaomar3 opened this issue 1 year ago • 5 comments

Added function to restore the old messages in chat module on submitting the correct assistant api key. Currently it is in testing phase, we have to add thread_id manually, otherwise it will start new thread.

The demonstration is attached below.

https://github.com/ArduPilot/MAVProxy/assets/106031961/2f63c47f-154e-4b4c-88cf-ec720af10a66

adityaomar3 avatar Mar 25 '24 06:03 adityaomar3

Next steps is to add a function to automatically store the thread id in the system and retrieve it next time onwards

adityaomar3 avatar Mar 25 '24 06:03 adityaomar3

This looks pretty good to me. As you say we need a way to either store the most recent thread's id in some persistent settings file or we need to be able to display a list of available threads to the user and let them pick one. Alternatively if we know which thread is the most recent we could just start using that one although it might be best to add a new menu item to "Continue from latest thread". Things could get interesting if one user has more than a single chat window open and tries to join the same thread from multiple chats but I think that's OK.

I've discussed with @peterbarker recently about whether there is a way to store settings but I don't think MAVProxy has this ability at the moment.

rmackay9 avatar Mar 26 '24 08:03 rmackay9

By the way, once we are closer to merging you might want to squash those two commits into a single commit.

rmackay9 avatar Mar 26 '24 08:03 rmackay9

Ping @adityaomar3 - are you still interested in pursuing this one?

peterbarker avatar Jun 10 '24 04:06 peterbarker

Hi @peterbarker, Yes! I will be working on this feature, currently I am working on some other features in chat module itself but this is on my list, I will surely address this but a bit later.

adityaomar3 avatar Jun 10 '24 21:06 adityaomar3