Skip to content
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

Make Chat Great Again #1414

Open
15 of 16 tasks
kodjima33 opened this issue Nov 26, 2024 · 26 comments
Open
15 of 16 tasks

Make Chat Great Again #1414

kodjima33 opened this issue Nov 26, 2024 · 26 comments
Assignees
Labels
bug Something isn't working Feature Request flutter flutter work

Comments

@kodjima33
Copy link
Collaborator

kodjima33 commented Nov 26, 2024

Describe the bug

Screenshots
Clipboard 25 Nov 2024 at 18 57
image

user ID (can we access the user info to validate the bug?):

--
refs:

@kodjima33 kodjima33 added bug Something isn't working Feature Request flutter flutter work labels Nov 26, 2024
@kodjima33 kodjima33 moved this to To do in omi TODO Nov 26, 2024
@josancamon19 josancamon19 self-assigned this Nov 26, 2024
@josancamon19
Copy link
Contributor

josancamon19 commented Nov 26, 2024

@kodjima33

  • (1) You mean that even if you don't have conversations related, it should provide a recommendation, right, based on your facts for example.
  • (4) Can you explain how do you give images/files, like what the chat would do with it, 1 example with each would help better understand this
  • (6) Not sure if I got it, you mean, that when you open the app, is not keeping track of the "plugin/app" used? Prob not, just tested this and it's working.

I think I got it, so you mean 3) 6) are related, so if we make each app to be an independent page, notifications should open accordingly, right?

@kodjima33
Copy link
Collaborator Author

  1. Yes
  2. Look how I interact with advisor when need help - send a file and ask question image

@josancamon19
Copy link
Contributor

@kodjima33 please test 1, 1.1 with your uid and questions you'd like

https://share.cleanshot.com/Pr4q2Klc

@josancamon19
Copy link
Contributor

@kodjima33 can we move (4) to another ticket, I feel like that's a separate project on it's own, so should be treated independently, also this are my thoughts on the implementation
Image

@josancamon19
Copy link
Contributor

  1. done, requires pushing the app first, otherwise if pushing the backend at the same time, otherwise chat plugins will not work.

@beastoin
Copy link
Collaborator

beastoin commented Dec 2, 2024

oops, anyway to make it backward compatible bro? or we have to wait for 70-100% current users update the new app.

@josancamon19
Copy link
Contributor

yea done

@josancamon19
Copy link
Contributor

But I cleared up all messages

@josancamon19
Copy link
Contributor

@josancamon19 josancamon19 removed their assignment Dec 3, 2024
@beastoin beastoin self-assigned this Dec 4, 2024
@beastoin
Copy link
Collaborator

beastoin commented Dec 4, 2024

bro @josancamon19 , did your merge all checked items to the main branch ?
(screenshot)

Image

@beastoin
Copy link
Collaborator

beastoin commented Dec 5, 2024

@mdmohsin7 hey man pls help me a bit with this ticket, use the current app version 1.0.48 (183) and play with it to verify if the above checked items are ready on current build or not.

@mdmohsin7
Copy link
Collaborator

Sure @beastoin let me

@mdmohsin7
Copy link
Collaborator

mdmohsin7 commented Dec 5, 2024

When I asked for chat recommendation, it gave me some stupid reply saying "i don't recall any conversations". See screenshot. It should actually help me

  • It worked initially for 1-2 messages and then it stopped making sense. If I ask for something, it finds completely unrelated memories for it
  • When I ask it for something (that is there in a memory), it continues with the previous message context, it does not respond to what I am asking at all (see screenshot)

I think it's time we divide chats. Let's make every chat with every app - separate. Not sure it's the best idea so if you'd have better proposals, lmk

  • Apps respond correctly but with unrelated memories
  • Apps respond correctly but also includes memories even when they are not needed
  • Apps send two initial messages instead of one
  • There are no user sent messages for the omi app if I close and open the main app
ScreenRecording_12-05-2024.13-58-50_1.1.mp4

@mdmohsin7
Copy link
Collaborator

The checked items are there in the app but are not fully functional or good enough to be used everyday

@beastoin
Copy link
Collaborator

beastoin commented Dec 5, 2024

feel free to pick one, take this serious.

complete it with your best then lmk.

@mdmohsin7 good luck 🚀

beastoin added a commit that referenced this issue Dec 7, 2024
Part of #1414 

- [x] There are no user sent messages for the omi app if I close and
open the main app
- [x] Apps send two initial messages instead of one
@beastoin
Copy link
Collaborator

beastoin commented Dec 7, 2024

by Joan


  • 1. When I asked for chat recommendation, it gave me some stupid reply saying "i don't recall any conversations". See screenshot. It should actually help me

  • 1.1 another stupid reply see screenshot

  • 3. I think it's time we divide chats. Let's make every chat with every app - separate. Not sure it's the best idea so if you'd have better proposals, lmk

  • 4. Need an ability to pass files/images. Every other AI chat can do it and it's cringe we still don't allow this Moved to it's own ticket

  • 6. Chat needs to remember the last app/persona i chatted with and open it. When user opens notification, make sure we open the correct chat

  • 2. in chatbox, instead of "ask friend anything" should be just "Message"

  • 5. Default chat should be with "omi" and always should be the priority. It's like 90% of users who chat with chatgpt default and not with apps.

  • When typing a question/message, the cat window needs to expand vertically. (if there is new line). Currently it doesn't expand vertically

  • EVERY MESSAGE in chat should be copyable. Even the message the user has sent. Now - not copyable. Pls fix it

@beastoin
Copy link
Collaborator

beastoin commented Dec 7, 2024

progress updates

(screenshot)

Image

@beastoin
Copy link
Collaborator

beastoin commented Dec 7, 2024

@beastoin
Copy link
Collaborator

beastoin commented Dec 8, 2024

add chat button to the app page

Image

@beastoin
Copy link
Collaborator

beastoin commented Dec 8, 2024

progress updates

(screenshot)

Image

beastoin added a commit that referenced this issue Dec 8, 2024
issue: #1414

## Deploy steps
- [ ] Deploy backend
- [ ] Deploy app
@beastoin
Copy link
Collaborator

@beastoin
Copy link
Collaborator

@gaulucky92

@beastoin beastoin mentioned this issue Dec 14, 2024
2 tasks
@beastoin
Copy link
Collaborator

progress updates
(screenshot)

Screenshot 2024-12-14 at 11 52 56

@beastoin beastoin moved this from In progress to In review in omi TODO Dec 15, 2024
beastoin added a commit that referenced this issue Dec 16, 2024
issue: #1414 


## Todos
- [x] Fix datetime filters when recalling the memory
- [x] Fix unrelated memories in chat response
@beastoin
Copy link
Collaborator

progress updates
(screenshot)

Image

@beastoin
Copy link
Collaborator

the remaining task will be updated at #1573

@beastoin
Copy link
Collaborator

Image
Image

@beastoin beastoin moved this from In review to Done in omi TODO Dec 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Feature Request flutter flutter work
Projects
Status: Done
Development

No branches or pull requests

4 participants