Skip to content

OpenNI2Device Add device sensor check for IR and depth modesetting - #2150

Merged
SergioRAgostinho merged 1 commit into
PointCloudLibrary:masterfrom
agrohn:openni2_viewer_fix
Dec 15, 2017
Merged

OpenNI2Device Add device sensor check for IR and depth modesetting#2150
SergioRAgostinho merged 1 commit into
PointCloudLibrary:masterfrom
agrohn:openni2_viewer_fix

Conversation

@agrohn

@agrohn agrohn commented Dec 14, 2017

Copy link
Copy Markdown
Contributor

Fix for issue #1348, tested with release 1.8.1 and latest master on Gentoo Linux, VTK-6.1.0 and VTK-7.1.0, OpenNI-2.2.0.33, latest libfreenect.

While it won't crash program should device lack IR and depth sensors, there is some peculiar stuff going on with threads; random crashes seem to appear upon startup.

@SergioRAgostinho

SergioRAgostinho commented Dec 15, 2017

Copy link
Copy Markdown
Member

Can you drebase to the current master please. We just fixed the problems with our CI.

@SergioRAgostinho SergioRAgostinho added the needs: code review Specify why not closed/merged yet label Dec 15, 2017
@taketwo

taketwo commented Dec 15, 2017

Copy link
Copy Markdown
Member

On Travis we don't have OpenNI2 anyway, so we don't need to see CI results.

@taketwo taketwo removed the needs: code review Specify why not closed/merged yet label Dec 15, 2017
@SergioRAgostinho
SergioRAgostinho merged commit ae4f57f into PointCloudLibrary:master Dec 15, 2017
@agrohn
agrohn deleted the openni2_viewer_fix branch December 15, 2017 11:13
@SergioRAgostinho SergioRAgostinho changed the title fixed issue #1348 by adding device sensor check for IR and depth modesetting OpenNI2Device Add device sensor check for IR and depth modesetting Aug 26, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants