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: Add non-interactive Docker environment for SWE-bench evaluation #1

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

fix: Add non-interactive Docker environment for SWE-bench evaluation #1

wants to merge 45 commits into from

Conversation

AlexCuadron
Copy link
Owner

@AlexCuadron AlexCuadron commented Feb 17, 2025 •
edited
Loading

This PR fixes the Docker build issue where it gets stuck on "Delete 1061 entries? (yes/no) [yes] Aborted" by:

  1. Adding a new docker_env.sh script that sets up non-interactive Docker environment
  2. Modifying the Docker environment to automatically handle confirmations
  3. Updating the pull script to use these non-interactive settings

The changes ensure that Docker commands run without requiring manual intervention while maintaining the same functionality.

Note: This PR has been updated to include the latest changes from upstream/main.

dependabot bot and others added 30 commits February 10, 2025 09:02
@dependabot @amanape
chore(deps): bump the version-all group across 1 directory with 3 upd…
61c709b 
…ates (All-Hands-AI#6648)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: amanape <[email protected]>
@tofarr
Removed in page callback (All-Hands-AI#6657)
707cb07
@malhotra5 @enyst @openhands-agent
[Bug fix]: Standardize SecretStr use (All-Hands-AI#6660)
4a5891c 
Co-authored-by: Engel Nyst <[email protected]>
Co-authored-by: openhands <[email protected]>
@rbren @openhands-agent @xingyaoww
Add comprehensive OpenHands glossary (All-Hands-AI#6310)
75f3f28 
Co-authored-by: openhands <[email protected]>
Co-authored-by: Xingyao Wang <[email protected]>
@malhotra5
[Enhancement]: Handle GH token refresh inside runtime (All-Hands-AI#6632
9bdc8dd 
)
@dependabot @amanape
chore(deps): bump the version-all group in /frontend with 4 updates (A…
6e35ac4 
…ll-Hands-AI#6665)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: amanape <[email protected]>
@malhotra5
[Resolver]: Add target branch param (All-Hands-AI#6668)
8688634
@neubig @openhands-agent
Fix issue All-Hands-AI#6262: Add success/failure indicators for file …
6c88b10 
…read/edit operations (All-Hands-AI#6653)

Co-authored-by: openhands <[email protected]>
@tofarr
Fix for issue where temp file is empty (All-Hands-AI#6669)
2b40a92
@neubig @openhands-agent
fix: Normalize whitespace when comparing patch context lines (All-Han…
7860055 
…ds-AI#6541)

Co-authored-by: openhands <[email protected]>
@fredysierra @enyst
fix: adding support for environment variables type dict (All-Hands-AI…
13839b4 
…#6672)

Co-authored-by: Engel Nyst <[email protected]>
@dependabot @enyst
chore(deps): bump docker/setup-qemu-action from 3.3.0 to 3.4.0 (All-H…
af0becd 
…ands-AI#6666)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Engel Nyst <[email protected]>
@dependabot @enyst
chore(deps): bump the version-all group across 1 directory with 9 upd…
a1107a2 
…ates (All-Hands-AI#6667)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Engel Nyst <[email protected]>
@amanape
hotfix: Typecheck routes during frontend build (All-Hands-AI#6676)
b12b426
@xingyaoww
Bump OpenHands ACI to 0.2.1 (All-Hands-AI#6678)
4615548
@enyst
Clean up global in llm.py (we figured it's not needed) (All-Hands-AI#…
1a715d2 
…6675)
@xingyaoww @openhands-agent
@enyst @neubig
feat(runtime): use prlimit to limit resource usage of command to av…
6a6dc93 
…oid OOM Runtime Kill (All-Hands-AI#6338)

Co-authored-by: openhands <[email protected]>
Co-authored-by: Engel Nyst <[email protected]>
Co-authored-by: Graham Neubig <[email protected]>
@xyeric
fix(frontend): fix public github repo cannot be selected (All-Hands-A…
6772227 
…I#6680)
@xingyaoww @openhands-agent
@neubig @enyst
refactor(runtime): Use openhands-aci file editor directly in runtime …
3188646 
…instead of execute it through ipython (All-Hands-AI#6671)

Co-authored-by: openhands <[email protected]>
Co-authored-by: Graham Neubig <[email protected]>
Co-authored-by: Engel Nyst <[email protected]>
@rbren
Fix debug in remote runtime (All-Hands-AI#6688)
1afe7f1
@dependabot
chore(deps-dev): bump @tanstack/eslint-plugin-query from 5.66.0 to 5.…
425ccc9 
…66.1 in /frontend in the eslint group (All-Hands-AI#6682)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@xingyaoww
refactor: do not add DEBUG env var when it is not set (All-Hands-AI#6690
a371562 
)
@amanape
Revert "Only show start project button in conversations" (All-Hands-A…
ff25e79 
…I#6698)
@xingyaoww
using all available system memory when RUNTIME_MAX_MEMORY_GB is not s…
f7c806c 
…et (All-Hands-AI#6691)
@tofarr
Fix log formatting error (All-Hands-AI#6699)
7e359ed
@amanape
chore: Throw a 404 instead of returning defaults if settings does not…
ba599c7 
… exist (All-Hands-AI#6704)
@malhotra5
Feat: Add selected branch param to backend (All-Hands-AI#6508)
312b9fb
@tofarr
Agent session no longer stuck in starting on raised exception (All-Ha…
cb5e7f0 
…nds-AI#6703)
@mamoodi
Release 0.24.0 (All-Hands-AI#6689)
653168f
@tofarr
More effective remote runtime identification (All-Hands-AI#6714)
d46d99a
csmith49 and others added 15 commits February 13, 2025 11:50
@csmith49
fix: Filter AgentCondensationObservation events from agent state (A…
341b695 
…ll-Hands-AI#6705)

Co-authored-by: Calvin Smith <[email protected]>
@dependabot @amanape
chore(deps): bump the version-all group across 1 directory with 5 upd…
b197e0a 
…ates (All-Hands-AI#6712)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: amanape <[email protected]>
@li-boxuan
Evaluation harness: Add agent config option (All-Hands-AI#6662)
ef12bc5
@wtiger9218 @josx
@Orianamartina @cuococarlos @juanmanueldaza @symbaventures @malhotra5 @enyst
feat(resolver): implement gitlab resolver (All-Hands-AI#6458)
f5fccab 
Signed-off-by: José Luis Di Biase <[email protected]>
Co-authored-by: José Luis Di Biase <[email protected]>
Co-authored-by: Oriana <[email protected]>
Co-authored-by: Charlie <[email protected]>
Co-authored-by: Juan Manuel Daza <[email protected]>
Co-authored-by: Juan Manuel Daza <[email protected]>
Co-authored-by: Cody Kociemba <[email protected]>
Co-authored-by: Rohit Malhotra <[email protected]>
Co-authored-by: Engel Nyst <[email protected]>
@malhotra5
fix: Simplify nested f-string to fix pydoc-markdown parsing (All-Hand…
edd5110 
…s-AI#6717)
@malhotra5
hotfix(Resolver): Workflow definition is out of sync with released pa…
85e3a00 
…ckage (All-Hands-AI#6719)
@amanape
feat(frontend): Settings screen (All-Hands-AI#6550)
0c03e25
@malhotra5
[Resolver]: Prep env in expectation of release (All-Hands-AI#6735)
1ddfa99
@ryanhoangt
chore: upgrade openhands-aci to 0.2.2 (All-Hands-AI#6731)
99b50d0
@dependabot @enyst
chore(deps): bump the version-all group across 1 directory with 12 up…
b07fddc 
…dates (All-Hands-AI#6736)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Engel Nyst <[email protected]>
@young010101 @enyst
docs: improve docstrings for CLI and config utils (All-Hands-AI#5398)
6356598 
Co-authored-by: Engel Nyst <[email protected]>
@li-boxuan
Add a sanity test for load_app_config and get_agent_config_arg (All-H…
efbff2e 
…ands-AI#6723)
@li-boxuan
A few fixes for TAC evaluation harness (All-Hands-AI#6586)
4443417
@kripper
Show docker build errors (All-Hands-AI#6695)
30e39e8
@AlexCuadron
fix: Add non-interactive Docker environment for SWE-bench evaluation
157fe37
@AlexCuadron AlexCuadron force-pushed the fix-docker-interactive branch from e8df22b to 157fe37 Compare February 17, 2025 02:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.