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

WIP - dmg/exe installable Letta #2072

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

norton120
Copy link
Collaborator

Please describe the purpose of this pull request.
This PR will add installable artifacts (dmg and exe) to the CI cycle. they can then be presented as "download Letta" links

How to test
In Phases:

  1. embedded pg: install from ./installable_apps/requirements.txt and then run ./installable_apps/startup.py. Visit http://localhost:8283 and create a new agent, verify that agent saves. Start and stop the service, confirm that the agent persisted.
  2. build installable for MacOS: Instructions TBD
  3. build installable for Windows: Instructions TBD
  4. CI release: Instructions TBD

@norton120
Copy link
Collaborator Author

Screenshot 2024-11-20 at 12 13 34 PM This is via the embedded Postgres

@norton120
Copy link
Collaborator Author

@norton120
Copy link
Collaborator Author

@norton120
Copy link
Collaborator Author

😠 In adjusting cutelog internals for our uses I am realizing that the QT license for OSS is dangerously restrictive for MemGPT/Letta's model. There's too much interpretation left open - so while having a native x platform log viewer is really nice out of the box, my opinion is that the risk is too high. So we'll need to roll our own simple log displayer. This is frustratingly not a solved problem (if it is and I'm missing it please share).

not a giant lift - if we write to rotating log files and then use one of the billion file parsers out there like (https://github.com/allproxy/json-log-viewer I've used before) or just dumping the log stream into a pretty/letta-branded SPA.
still frustrating.

@norton120
Copy link
Collaborator Author

@norton120
Copy link
Collaborator Author

running python3 setup.py py2app with a clean venv will build a dist that can be directly archived into a dmg image. But running the dmg image faults because there are missing deps. Installing those deps beforehand (pip3 install -r requirements.txt) introduces a recursion issue somewhere via ast - I think this issue is in llama index, just from process of elimination.

the sys.setRecursionLimit hack gets it to build but fails with missing llama_index even when it's been manually installed - so my theory is that llama_index is failing silently with the hack.

from here I need to try and find a workaround for the recursion issue, and then we have a working dmg installable 👍🏻

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

Successfully merging this pull request may close these issues.

1 participant