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

Temp. hack -- adding depth registration code missing from the latest freenect driver. #88

Merged
merged 4 commits into from
May 14, 2015

Conversation

Algomorph
Copy link
Contributor

This is directly related to #82.
I had no clue this was apparently being done already by Dorian himself in #63. Anyway, this is a temporary fix for that that uses exactly the same code as dorian3d's libfreenect2 fork for depth registration and seems to have no merge conflicts. We can use this until #63 is merged.

@arpg-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

@crheckman
Copy link
Contributor

I've dusted off #63 and it is ready to be merged into master, which if I understand you correctly will mean that this PR can be closed. Does #63 solve your issues?

@Algomorph
Copy link
Contributor Author

I'll try it out and let you know, thanks.

@Algomorph Algomorph closed this May 11, 2015
@crheckman crheckman reopened this May 14, 2015
@arpg-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

crheckman added a commit that referenced this pull request May 14, 2015
This is a reasonable solution until we re-port the freenect2 driver.
@crheckman crheckman merged commit 73ce13d into arpg:master May 14, 2015
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