GuanxingLu / ManiGaussian

[ECCV 2024] ManiGaussian: Dynamic Gaussian Splatting for Multi-task Robotic Manipulation
MIT License
150 stars 5 forks source link

Error in training #11

Closed cheng052 closed 1 month ago

cheng052 commented 1 month ago

Hi Guanxing,

Thx for your good work!

I have tried to run the train script bash scripts/train_and_eval_w_geo_sem_dyna.sh ManiGaussian_BC 0,1 12345 ${exp_name}, but encountered an importError.

Have you ever encountered such ImportError in mask2former? Glad to be heard!

Screenshot from 2024-07-23 21-57-13

Regards, Bowen

kjeiun commented 1 month ago

That problem can be fix with ERORR_CATCH.md

cheng052 commented 1 month ago

That problem can be fix with ERORR_CATCH.md

I have checked the ERORR_CATCH.md but not found the fix to ImportError, maybe you can point out it more clearly?

regards, Bowen

kjeiun commented 1 month ago

Oh i was confused with build error, sry! BTW, are you running inside of a docker? Because i’m having troble with RLBench for generating demonstrations running inside of a docker :(

cheng052 commented 1 month ago

Oh i was confused with build error, sry! BTW, are you running inside of a docker? Because i’m having troble with RLBench for generating demonstrations running inside of a docker :(

I’m running the code on a physical Ubuntu workstation instead of a docker.

GuanxingLu commented 1 month ago

@cheng052 Sorry for the delayed response. I have not encountered this issue before. Perhaps you can check to see if there is any file with the same name as 'mask2former'?

kjeiun commented 1 month ago

Did you solve it? I've encountered the same problem :(

kjeiun commented 1 month ago

I think this problem is related to the version of hydra

1) Hydra version==1.1.0 then assert mdl is not None error occurs.

2) Hydra version==1.1.1 then mask2former circular problem occurs.

As following the Install.md manual, I installed hydra-core==1.1 (=1.1.0) but this makes the assertion error. Do you have any idea? Should I install new version of mask2former?

I'm using cuda 11.7 ubuntu22.04 with nvidia GPU L40S :(

GuanxingLu commented 1 month ago

Sorry for the late reply, I also have no idea with this import problem... But if you do not need the sem. loss, you can just comment out the odise package in ManiGaussian, and it will not affect the geo. and dyna. losses.

kjeiun commented 1 month ago

I see. Thanks

Can you upload the installed libraries(pip list/ conda list) for ubuntu 22.04 / cuda 11.7?

GuanxingLu commented 1 month ago
(manigaussian) guanxing@ps:~$ nvcc -V
nvcc: NVIDIA (R) Cuda compiler driver
Copyright (c) 2005-2022 NVIDIA Corporation
Built on Tue_May__3_18:49:52_PDT_2022
Cuda compilation tools, release 11.7, V11.7.64
Build cuda_11.7.r11.7/compiler.31294372_0
GuanxingLu commented 1 month ago

condalist.txt piplist.txt Here are the exported files.

cheng052 commented 1 month ago

@kjeiun @GuanxingLu Error fixed. The ImportError bug is due to two "data" folders being missing in the repo. Just copy GNFactor/third_party/ODISE/odise/data, GNFactor/third_party/ODISE/third_party/Mask2Former/mask2former/data to the corresponding location in ManiGaussian, and reinstall odise / mask2former / hydra-core==1.1.0.

It seems like the "data/" item in .gitignore also ignores these two folders in the third_party folder, which is strange.

kjeiun commented 1 month ago

Oh it works as you said. Thanks.

GuanxingLu commented 1 month ago

@kjeiun @GuanxingLu Error fixed. The ImportError bug is due to two "data" folders being missing in the repo. Just copy GNFactor/third_party/ODISE/odise/data, GNFactor/third_party/ODISE/third_party/Mask2Former/mask2former/data to the corresponding location in ManiGaussian, and reinstall odise / mask2former / hydra-core==1.1.0.

It seems like the "data/" item in .gitignore also ignores these two folders in the third_party folder, which is strange.

Thanks for pointing out this!