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

Fixed missing virtual function #5

Open
wants to merge 1 commit into
base: unstable
from

Conversation

Projects
None yet
6 participants
@ph4m

ph4m commented Oct 24, 2013

Updated XnSensorDepthGenerator with dummy function GetPixelCoordinatesInViewPoint for compatibility with latest OpenNI virtual functions.

@Batistleman

This comment has been minimized.

Show comment
Hide comment
@Batistleman

Batistleman Jan 7, 2014

thanks!

can this be merged?

Batistleman commented Jan 7, 2014

thanks!

can this be merged?

@hustcalm

This comment has been minimized.

Show comment
Hide comment
@hustcalm

hustcalm Dec 31, 2014

@Batistleman
I verified the patch and it works under my Ubuntu 12.04 box.

hustcalm commented Dec 31, 2014

@Batistleman
I verified the patch and it works under my Ubuntu 12.04 box.

Avnerus added a commit to Avnerus/SensorKinect that referenced this pull request Oct 27, 2015

@alidaf

This comment has been minimized.

Show comment
Hide comment
@alidaf

alidaf Jan 5, 2018

I spent nearly a week trying to install this using the outdated instructions. Finally applying this patch allowed it to work.

alidaf commented Jan 5, 2018

I spent nearly a week trying to install this using the outdated instructions. Finally applying this patch allowed it to work.

@UneasierSine

This comment has been minimized.

Show comment
Hide comment
@UneasierSine

UneasierSine Jan 28, 2018

This was a really big help. I spent five hours looking through forums as to why there was no Redist directory. Now I was able to fix and install the rest of the library,

UneasierSine commented Jan 28, 2018

This was a really big help. I spent five hours looking through forums as to why there was no Redist directory. Now I was able to fix and install the rest of the library,

@JustinMMiller

This comment has been minimized.

Show comment
Hide comment
@JustinMMiller

JustinMMiller Apr 7, 2018

This patch just saved me another 6+ hours of attempting compilation. Thank you so much!

JustinMMiller commented Apr 7, 2018

This patch just saved me another 6+ hours of attempting compilation. Thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment