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

feat: improve CLI appearance #687

Merged
merged 4 commits into from
Dec 23, 2023
Merged

feat: improve CLI appearance #687

merged 4 commits into from
Dec 23, 2023

Conversation

cpacker
Copy link
Collaborator

@cpacker cpacker commented Dec 23, 2023

Please describe the purpose of this pull request

  • Patches the annoying llama-index "MockLLM" message that's confusing to new users
  • Adds extra spacing + changes color schemes to make CLI look nicer

How to test

memgpt run with a non-empty agents dir

image image

memgpt run with empty home dir

image image

@cpacker cpacker requested a review from sarahwooders December 23, 2023 08:07
# Supress llama-index noise
with suppress_stdout():
# TODO: allow configrable state manager (only local is supported right now)
persistence_manager = LocalStateManager(agent_config) # TODO: insert dataset/pre-fill
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Moved persistence_manager assignment after - @sarahwooders I don't think this changes anything though? Can also move back (was just moving it around to ID where the stray MockLLM print statement was coming from)

@cpacker cpacker merged commit 19c4891 into main Dec 23, 2023
3 checks passed
@cpacker cpacker deleted the pretty-print branch December 24, 2023 08:35
sarahwooders pushed a commit that referenced this pull request Dec 26, 2023
* prettified standard cli output

* also suppress stdout for agent loading

* better spacing

* added emoji to existing agent load
norton120 pushed a commit to norton120/MemGPT that referenced this pull request Feb 15, 2024
* prettified standard cli output

* also suppress stdout for agent loading

* better spacing

* added emoji to existing agent load
mattzh72 pushed a commit that referenced this pull request Oct 9, 2024
* prettified standard cli output

* also suppress stdout for agent loading

* better spacing

* added emoji to existing agent load
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