fix(backend) changes to improve Command-R+ behavior, plus file i/o error improvements, attempt 2 #1417
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…ous file IO errors, added timeout and max return token configurations for the LLM api.
…sandbox when resolving paths in fileio operations, add customizable timeout for bash commands, mention said timeout in llm prompt.
…ng to delete it from the response afterwards, fixed get_working_directory for ssh_box.
…r-plus-changes # Conflicts: # opendevin/action/fileop.py
… command-r-plus-changes # Conflicts: # agenthub/monologue_agent/utils/prompts.py # opendevin/llm/llm.py # opendevin/schema/config.py # tests/test_fileops.py
… command-r-plus-changes # Conflicts: # opendevin/config.py
|
This integration test failure does not occur locally for some reason, investigating. |
enyst
reviewed
Apr 28, 2024
…n local sandbox mode, combine exception handlers in prompts.py.
|
@rbren I have fixed the integration test issue. It was introduced when the local-sandbox integration test was added late last night 7d5856e . The test environment configuration did not account for how the path inside the sandbox could no longer be /workspace, and had to be adjusted to match the external host path instead. |
This was referenced Apr 28, 2024
enyst
approved these changes
Apr 28, 2024
|
I'm sorry about that @computer-whisperer , I looked at the issue and missed that conflict. I think we could get this one in, it has some pretty awesome changes and they seem to help even poor old GPT-3.5! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
#1409