Skip to content
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

Color integration #15

Open
wants to merge 21 commits into
base: master
Choose a base branch
from
Open

Color integration #15

wants to merge 21 commits into from

Conversation

gcuendet
Copy link

I propose to integrate colors to kinfu_remake. Similarly to what is available in PCL kinfu, here is the color integration and color point cloud generation part. As a result, the point cloud now contains color information. The following two figures show a comparison of the same scene:

  • Without color integration
    screenshot_kinfu
  • and with color integration
    screenshot_kinfu_colors

The code is work in progress and might not be fully optimized. Let me know what you think about it.

@Nerei
Copy link
Owner

Nerei commented Jul 12, 2016

Thanks. Quite large patch to review it quickly.

@gcuendet
Copy link
Author

Hello. I also ported marching cube from old PCL kinfu, as requested in issues #10 and #12. Moreover, the colors are integrated to the generated mesh (in the same way as for the cloud). Here is a screenshot example of the results:
kfusion_color_mesh

Currently, these changes are on a separate branch feature/MarchingCube. I was thinking maybe I should wait for you to review this pull request before I open a new one with the MarchingCube, what do you think?

Also, let me know if I can help in any way with the pull request or anything else ;)

@ghost
Copy link

ghost commented Apr 28, 2017

Hi @gcuendet

Is what I can calculate tsdf for a cloud of points not for depth images ?
You can help me with an idea

Thank you for your help.

@447425299
Copy link

@gcuendet Thanks for you great job, And I have build your project successfully, but i do not know how to run it. What is the Incoming parameters?Thank for your help.
image

@447425299
Copy link

@PlayerUsr Could you tell me how to run the project, @gcuendet build. what is the argv[]?
image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants