vdbfusion
vdbfusion copied to clipboard
Color support
Adding support for colors mostly with work from @swarmt but I added the last piece which is marching cubes support.
This is an example from Lady and Cow: Image: https://drive.google.com/file/d/1LF35dqW4x5z_Fu2rQ-UFAJuwukVjfsPn/view?usp=sharing Mesh: https://drive.google.com/file/d/1OiCMfb4HFpLIdbGzFEW7ZGTqLMBxbomL/view?usp=sharing
This PR is related to https://github.com/PRBonn/vdbfusion_ros/pull/1
Wow! @paucarre thanks a lot for opening the PR!
I will allocate some time to deep-analize the PR before merging, but over the surface looks quite good :)
@paucarre I started to review this, is there any chance you run clang-format on your end? Seems like the format has changed a lot, and makes the reviewing a bit challenging.
You can run this command on the root of the project and that would do the magic:
clang-format -i $(find . -regextype posix-extended -regex ".*\.(cpp|cxx|cc|hpp|hxx|h)" | grep -vE "^./(build|3rdparty)/")
@nachovizzo I will update the PR this (late) evening
@nachovizzo updated. Note that I forced C++17 as it seems it no longer builds unless it uses C++14. I think C++17 has advantages so if you force a a C++ maybe leave at least C++17
@paucarre thanks for the PR, I tested the code and it works fine. The only thing is that you did not multiply the color values by 255 at the end. Not sure if you did this intentionally.
@melhashash Thanks for giving this a try. I will review this next week and merge it, so it's also accessible through the python packages
@melhashash Thanks for giving this a try. I will review this next week and merge it, so it's also accessible through the python packages
Can you please review and merge it?
Hello @vnmsklnk @paucarre . I'm honestly sorry for the delay. I currently had no time for reviewing this.... and sadly no one else can do it for me :(....
So this will have to wait in the meantime. You can always use the @paucarre and build from source in case you need it ;_)
@paucarre ,Hi, can you give me some advice on how to download the cow datasets?
Excellent, thanks for this. Thought I may need to implement it myself.
@paucarre I'd fork from your repo and add my changes on top if you don't mind. One of the reasons is that the entire CI infrastructure for this project is not running on Github but in our private Gitlab servers, therefore I need to make sure all checks and tests are passing
Will merge this in #33 . Thanks @paucarre