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

Support for other RGB-D Hardware? #14

Open
NicksonYap opened this issue Dec 10, 2018 · 5 comments
Open

Support for other RGB-D Hardware? #14

NicksonYap opened this issue Dec 10, 2018 · 5 comments

Comments

@NicksonYap
Copy link

NicksonYap commented Dec 10, 2018

Hi,

I have Kinect 1, but have yet to try your code

It seems that Asus Xtion was used in your paper and officially supported.

The best Intel Realsense, the D435 model is going for 180 USD and seems to much better specs than the Asus or Kinect

It'll be awesome if Co-fusion supports Intel Realsense

Any suggestion how the Intel Realsense (or other sensors) should be supported?
I may purchase one and submit a PR

Thanks!

@NicksonYap NicksonYap changed the title Recommended RGB-D Hardware? Other supported RGB-D Hardware? Dec 10, 2018
@NicksonYap NicksonYap changed the title Other supported RGB-D Hardware? Support for other RGB-D Hardware? Dec 10, 2018
@511963041
Copy link

I found that -- https://github.com/Daichou/ElasticFusion/tree/rs2 -- has the realsense support for ElasticFusion. For Co-fusion is based on ElasticFusion, so the input stream is same. I changed a few codes and it worded well with realsense D435.

My environment:
win10 x64
vs15 x64 release
cuda 9.0

@NicksonYap
Copy link
Author

NicksonYap commented Dec 22, 2018

@511963041

Great find! although ElasticFusion seems to already support RealSense natively

Can you share the commits here?

Would be great to see your code, or even submit a PR

@511963041
Copy link

run -- https://github.com/Daichou/ElasticFusion/tree/rs2 -- by @Daichou first,
compare the "logreader" in MainController.cpp between ElasticFusion(@Daichou ) and co-fusion.

As for my codes, I add some like - ini file read for rs settings- and -absolute paths-,replace may cause new problems.

Another suggest:
This branch -- https://github.com/Daichou/ElasticFusion/tree/rs2 -- does not add align process for Realsense, you should add that.

@Daichou
Copy link

Daichou commented Jan 14, 2019

hi @511963041, I add align proccess to https://github.com/Daichou/ElasticFusion/tree/rs2 this branch. Please take a look and give me some suggestion if you have time.

@ttsesm
Copy link

ttsesm commented Apr 12, 2019

@NicksonYap @Daichou did you manage to run co-fusion with realsense. Is it by any chance possible to provide some hints what is needed?

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

No branches or pull requests

4 participants