-
-
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. |
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: