-
Notifications
You must be signed in to change notification settings - Fork 8
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
waypoint_type etc. None in waypoints_info in cliport_test #4
Comments
Thanks for reaching out. The None parts of waypoint information are used for sub-step evaluations, and we do not save this information in the current validation sets (The validation sets were generated previously and they are time-consuming due to the rendering). Therefore, there are two ways to fix this:
|
Thanks for the detailed response @KzZheng ! However, validation is showing 0% success. And it sometimes says "Could not create path." Do you know what might be off? Thanks, |
I re-download the unseen validation set for picking, and I found I could run the test on this task without any code change. The results are also not zeros. Can you check whether you are using the correct weights and dataset? Thanks! PS: "Could not create path" is normal since the estimation errors will cause the failures of path planning. You can try to set "--ignore_collision" as True to ignore the collisions. |
The issues are fixed by copying the Lua file into the Coppliasim folder. |
waypoint_type missing (code complains need re-generate) turns out to be because I didn't copy over the lua script. There is also a gdrive download issue tracked here: prasmussen/gdrive#645 |
Hi, I'm also tying to get this code to work, but how did you manage it?
Are the test data I downloaded wrong? Thanks, |
Hi Daichi @daichisaito-cs, Le fixes the error by copying the Lua file (https://github.com/eric-ai-lab/VLMbench/blob/main/simAddOnScript_PyRep.lua) into the Coppliasim folder. I'm not sure whether you have the same issues. Best, |
Thanks for quick reply, I tried copying that file into the Coppliasim folder, but it didn't work and I got the error 'need re-generate ~'. |
There are only two potential issues that will lead to 'need re-generate' in my mind:
I think you should be one of these cases. |
You can also add
Under the exception VLMbench/vlm/scripts/cliport_test.py Line 329 in dfc3e53
|
I already copied lua file, so it might be the first issue you mentioned. Is the following link right for downloading the test_seen data? |
The link is correct. Can you confirm you use the Coppeliasim 4.1? |
Yes, I use the Coppeliasim 4.1.0. |
Can you add this code to the file to provide more error details? |
The error I got when I added traceback was like this:
|
I just test the code. Every thing works fine for me. Can you add the following code block
before VLMbench/vlm/scripts/cliport_test.py Line 306 in dfc3e53
To see whether you can successfully write and read metadata from coppeliasim? |
When I run the picking task, I got the following error.
When I run the drawer task, I got this error.
|
I'm not sure about the first error. But at least the second error indicates that you are not using the correct Coppeliasim and Lua files. Please check the PyRep's install steps to see whether you have finished all the steps, e.g., the path to Coppellasim:
And simAddOnScript_PyRep.lua should be directly copied to the CoppeliaSim's main folder after PyRep installment. |
The first error is caused by typos in the code. You can change the test code block to
And then you should see the same issue across all tasks. |
I see, the cause is now clear. |
Sure. You can check whether the function |
Hi @KzZheng , I tried doing PyRep's install steps again, but it's still not working yet. I put PyRep and CoppeliaSim directories below VLMbench directory, but is that correct? Thanks |
I do not put PyRep and CoppeliaSim directories below the VLMbench directory. I downloaded them in two separate folders. PyRep read the environment variable Also, if you have a monitor, you can set |
I could run the example in the PyRep with CoppeliaSim visualization. |
I finally managed to solve the problems! It appears that simAddOnScript_PyRep.lua file within VLMbench was somehow overwritten with the original PyRep.lua file. So the command "cp ./simAddOnScript_PyRep.lua /Path/To/CoppeliaSim" did not copy the correct content. I'm actually considering using VLMbench for my research on manipulation tasks since I believe it's a highly useful benchmark. I really appreciate your help. |
Good! Thank you for your interest in this project! |
Hi VLMBench gurus,
I'm just trying to run the cliport_test script with the provided models.
I installed the code and dependencies, and downloaded the trained model, and the seen and unseen splits of data:
bash download_dataset.sh -s ~/vlmbench/Dataset -p valid_unseen -t pick
bash download_dataset.sh -s ~/vlmbench/Dataset -p valid_seen -t pick
When running cliport_test, waypoint_type etc. fields are all None in waypoints_info array.
python vlm/scripts/cliport_test.py --task pick --data_folder /Path/Dataset/valid --checkpoints_folder /Path/models
This is the waypoints_info array:
['waypoint0', None, None, 1, None, False, array([ 0.09891216, -0.09790423, 0.85907155, -0.64400905, 0.76425868,
-0.01948901, 0.02795067])], ['waypoint1', None, None, 1, None, False, array([ 0.10433818, -0.0974073 , 0.7792573 , -0.64400905, 0.76425868,
-0.01948901, 0.02795067])], ['waypoint2', None, None, 1, None, False, array([ 0.09755566, -0.09802846, 0.8790251 , -0.64400905, 0.76425868,
-0.01948901, 0.02795067])], ['waypoint3', None, None, 1, None, False, array([ 0.43130848, -0.15798603, 0.85353625, 0.39130512, 0.79267031,
-0.0298808 , 0.46654186])]
The code catches this error and says "need re-generate: /Path/valid/seen/pick_cube_shape/variation0/episodes/episode4".
Am I missing some installation step? Am I downloading the correct dataset files?
Thanks,
Le
The text was updated successfully, but these errors were encountered: