-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
[Bug]: [SWE-bench] Faild to cd. ModuleNotFoundError #231
Comments
specific to OpenHands version 0.18.0 |
Could you let me know why you're using that version? |
I’ve been using the 0.18.0 version since its release. However, I recently switched to version 0.21.0 and encountered an issue.
|
Would you run |
I couldn't find the specific container id from the issue, and after running docker ps -a, there are many containers I found. So it is not feasible to check each of them. However, I found that this error occurred if I am running more than one worker. For a single worker, it works fine. I think this issue occurred if there are parallel workers. Another note: For parallel workers there is another kind of issue i noticed-
I changed the sandbox_config.py according to the comment mentioned in a different issue thread. The following is the current version-
|
First issue: another container may use the same port and already started and it checks the folder in another container. Root cause: when running multiple workers, same port is being used. |
I will try to provide custom ports in the code section you mentioned. Thank you. Apart from this issue, I am facing another issue- Using openhands 0.21.0 version
|
Thanks for your earlier response. Could you please provide more details on how to address this |
It's due to the port conflict. Resolution for both issues: Unique port for each instance. |
Snippet to map each port. from datasets import load_dataset
dataset = load_dataset(
'princeton-nlp/SWE-bench_Lite',
cache_dir='./cache',
verification_mode='no_checks',
num_proc=4,
split='test',
)
port_range = 63000
port_mapping = {}
for i in range(len(dataset)):
port_mapping[dataset[i]['instance_id']] = port_range + i
print(port_mapping) |
Did you check about swebench_verified_mini? |
Would you apply this commit and check if the issue resolves for you? |
I’m applying this commit right now. I’ll keep you updated if the issue is resolved. |
I applied the commit, but it didn’t resolve the “failed to cd” issue. Currently, I have two issues:
|
Now this is the new container name. Would you check the container logs? Would you apply this commit to see why the buildx command failed? |
-> There is no such container in this name in the container list. I believe the buildx command failed because it couldn’t locate the container, as mentioned in the inference logs. Interestingly, when I rerun the inference process for that specific instance, it completed without any issues. However, the same issue persisted for different instances later on.
I applied this commit and reran the inference process. |
|
There are two issues now and these issues occurred on multiple instances-
Running openhands again and will share the logs with you. |
Would you run only these two instances simulataneously? |
In this time, they ran without any problems. However, the issue of “failed to cd” still persists. After applying the mapping port commit you provided, it generates only empty patches. This commit The normal steps begin at 1 and then 2, 3, 4, and so on, up to 30. However, for this commit, it starts randomly, such as 13, performs 0 to 5(n) steps, and then generates empty patches. |
Is the llm |
128k |
For temperature 0, I checked using this commit and without it. Without it, there a non-empty patch generated, but when I use this commit, there’s an empty patch. |
Even when running a single instance, did it produce an empty patch? |
Yes, I checked running a single instance and it produce an empty patch. Is this commit worked fine from your side? |
Would you provide the console logs? |
Did you check the remote runtime? |
No. After applying this commit. Can you please just verify this commit and update it if necessary. Because after applying, it produce empty patches. Specially for the following line of codes from your commit I think (Maybe I am wrong here)-
You can see from the below logs, it start at step 13 randomly I think.
|
Do you want to create the container for each run? Else, Lines 95 to 96 in 9fa0e6b
|
Are you using the seed parameter? |
No
Can you please provide the code Snippet for this? Thanks!!!
You are referring these two lines in setup.py. Am I correct? |
Would you check this commit ? (not tested yet)
yes. |
Checked but it didn't work. Same result as above. --
Could please check how we can achieve this? Mapping port for instances is great idea and hopeful that it will solve |
One thing I noticed- I didn't use the port mapping, just ran the openhands using 4 workers and faced
|
In the last week's error itself, it was failed due to the previous command running. Would you apply this commit only and check? Related discussion #118 |
All-Hands-AI#6431 Issue
@BIJOY-SUST, Is this issue specific to OpenHands version 0.18.0, as it's not the latest?
The text was updated successfully, but these errors were encountered: