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: add argument --base-container-image to resolver #7612

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

Conversation

Tetsu-is
Copy link

@Tetsu-is Tetsu-is commented Apr 1, 2025

  • This change is worth documenting at https://docs.all-hands.dev/
  • Include this change in the Release Notes. If checked, you must provide an end-user friendly description for your change below

End-user friendly description of the problem this fixes or functionality that this introduces.

Enabled custom sandbox for resolver. You can apply base image with --base-container image YOUR_BASE_CONTAINER_IMAGE

example

python -m openhands.resolver.resolve_issue --selected-repo [OWNER]/[REPO] --issue-number [NUMBER] --base-container-image [YOUR_BASE_CONTAINER_IMAGE]

Give a summary of what the PR does, explaining any non-trivial design decisions.
I added base_container_image setting for resolver in order to enable github actions to use any programming language.
Instead, we have to specify -runtime-container-image even when we don't use custom sandbox(I will fix it so that user dont have to specify when using defaultt runtime image)

  • add command line argument --base-container-image to resolver.resolve_issue.py
  • add argument line--base-container-image to resolver.resolve_all_issues.py
  • without --runtime-container-image, set runtime_container_image=None and build sanbox with base_container_image

Link of any specific issues this addresses.

I haven't created issue about this topic.

@Tetsu-is Tetsu-is changed the title add argument --base-container-image for resolver feat: add argument --base-container-image for resolver Apr 1, 2025
@Tetsu-is Tetsu-is changed the title feat: add argument --base-container-image for resolver feat: add argument --base-container-image to resolver Apr 1, 2025
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