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

Issue in 'event' tag for adding memory #2043

Open
microcoder-py opened this issue Nov 22, 2024 · 2 comments
Open

Issue in 'event' tag for adding memory #2043

microcoder-py opened this issue Nov 22, 2024 · 2 comments

Comments

@microcoder-py
Copy link

microcoder-py commented Nov 22, 2024

🐛 Describe the bug

Regards to #2030, the error ERROR:root:Error in new_memories_with_actions: 'event' is stemming from mem0/memory/main.py function _add_to_vector_store

The LLM is not generating the correct JSON output, i.e. does not have specific key

However, this should not cause an error. We should be able to manually provide event, or at least provide a default value in case of error

@microcoder-py
Copy link
Author

@Dev-Khant when using locally hosted LLMs, the prompt is too long for the LLM to analyse it and correctly respond. This is especially true of get_update_memory_messages

This is causing missing keys for events, and incorrect results (it basically updates nothing, retaining the same memory)

Can we shorten the prompt?

@rajesh0854
Copy link

Having the same issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants