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.

Sign up for GitHub

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

Jump to bottom

Fix Issues for App #275

Open
wants to merge 21 commits into
base: main
Choose a base branch
from
Open

Fix Issues for App #275

wants to merge 21 commits into from

Conversation

dillonalaird
Copy link
Member

@dillonalaird dillonalaird commented Oct 16, 2024 •
edited
Loading

Fixes some bugs:

  • Fixes a bug on switching between new and old format
  • Fixes bug where json extract was raising ValueError and not getting caught properly
  • Fixes bug where file names were getting added to the artifact but they didn't exist
  • Changes media extraction to use media from ipython display call
  • Fixes bug with edit_code_artifact where VA was using -1 as line number. Also adds an example of how to use edit_code_artifact in VA prompt
  • Removes verbosity env arg which was taking up extra token space in the VA chat observations and limits "viewable" conversation to 10 messages to keep tokens lower
  • Updates prompts on how to deal with artifacts, "user can see artifacts", "artifacts are saved only after execution is finished", etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yzld2002 yzld2002 yzld2002 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dillonalaird @yzld2002