drprojects / DeepViewAgg

[CVPR'22 Best Paper Finalist] Official PyTorch implementation of the method presented in "Learning Multi-View Aggregation In the Wild for Large-Scale 3D Semantic Segmentation"
Other
222 stars 24 forks source link

Python 3.8.5 #32

Closed adikrishnanucd closed 1 year ago

adikrishnanucd commented 1 year ago

@drprojects What changes would I need to make to the install.sh and/or the deep_view_aggregation.yml to set up the conda environment with Python 3.8.5 instead of 3.7.9?

drprojects commented 1 year ago

Hi, you could try setting:

  - python=3.8.5 

in here.

But, due to some dependencies limbo between Torch-Points3D, PyTorch Geometric, and Torch, I cannot vouch for the success of installing with such approach.

adikrishnanucd commented 1 year ago

I will try this. Thank you @drprojects.

drprojects commented 1 year ago

I would be curious to know if that works. Please keep let me know how that goes.

adikrishnanucd commented 1 year ago

There seemed to be a few errors when trying to run it with Python=3.8.5. Instead I am trying to run it with Python=3.7.9 as in the original .yml file. Is there anything I have to do to run the kitty-360 inference notebook through Jupyter notebook within VSCode. I have set the kernel to be the deep_view_aggregation conda environment. But for some reason it crashes when the second cell runs. I am getting this error @drprojects :

Disposing session as kernel process died ExitCode: undefined, Reason: ~/.local/lib/python3.8/site-packages/traitlets/traitlets.py:2395: FutureWarning: Supporting extra quotes around strings is deprecated in traitlets 5.0. You can use 'hmac-sha256' instead of '"hmac-sha256"' if you require traitlets >=5.
  FutureWarning,
~/.local/lib/python3.8/site-packages/traitlets/traitlets.py:2349: FutureWarning: Supporting extra quotes around Bytes is deprecated in traitlets 5.0. Use 'a8252496-ab37-400c-9a9f-b15955432e56' instead of 'b"a8252496-ab37-400c-9a9f-b15955432e56"'.
drprojects commented 1 year ago

This seems to be a VSCode-Jupyter issue: https://github.com/microsoft/vscode-jupyter/issues/9347

I can't help with that one, you will have to figure this out !

Best,

Damien

adikrishnanucd commented 1 year ago

@drprojects Would it be possible to run the kitti360_inference notebook as a Python script within the deep_view_aggregation conda environment? I am having trouble running it as a notebook, so I am thinking of copying the code over to a python script.

drprojects commented 1 year ago

Unless you are using VSCode to connect to a remote conda install, I am not sure as per why you are using VSCode to run a notebook. Can't you rather launch a jupyter lab or jupyter notebook from your bash and open the notebook in your browser ?

Of course, the notebook is just a tool for people to get started. You can definitely write your own scripts. You can also get inspiration from the bash scripts in scripts/.

But overall, the best way to get familiar with the project is to look into the code (which is abundantly commented) and read the paper :wink:

adikrishnanucd commented 1 year ago

@drprojects I am using VSCode to connect to a remote conda install which is also a remote server where the code lies in. I will continue to try to run Jupyter lab directly, as running the Jupyter notebook from within VSCode continues to give me an error. On a first try, I have been having the kernel crash and restart when I try to run the second cell of the kitti360_inference notebook through Jupyter lab. I am trying to figure out why this is the case.

Thank you.