-
Notifications
You must be signed in to change notification settings - Fork 110
Fix bug in visualize.py
and add Open3D-based visualization
#87
Conversation
Thanks for the contribution! The newly added visualizer is not used right now. Is |
visualize.py
and add Open3D-based visualization
54674dc
to
53a281c
Compare
hello,zhijian, |
Hello, @suyunzzz
But I encounter some error about:
The parse argument is:
Do you know how to solve this problem? |
Hi @suyunzzz, thanks for the updates. Could you please remove |
Btw, could you also reformat the code according to |
@QuasarsZ, could you please double-check the input size: |
Hello, @zhijian-liu. Thanks for your response. I run the visualize code with:
And the input size is:
Is my velodyne-dir set incorrectly? |
Does it work if you use a original visualize.py? |
|
Hi, @suyunzzz. Thank's for your reply. Run original visualize.py will get another error. (#90) Then, I only modify this:
to
And I will get error message about:
|
@suyunzzz To use precommit,
It should reformat the code and then format the code before every commit. |
thanks, it seems work😄 |
add open3d backend for visualization reformat visualize.py
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks really nice @suyunzzz. Thank you for your great efforts!
Fix a bug in
visualize.py
and add Open3D-based visualization.