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

Better Installation instructions #4

Open
IshankGulati opened this issue Feb 28, 2016 · 3 comments
Open

Better Installation instructions #4

IshankGulati opened this issue Feb 28, 2016 · 3 comments

Comments

@IshankGulati
Copy link
Contributor

Installation instructions on opendetection homepage is a bit lacking.

OpenNI is an optional dependency in PCL and OpenCV but opendetection doesn't build without it as it requires BUILD_apps_3d_rec_framework=ON. It should be listed as an additional dependency.

Also, in my case I had to use BUILD_TESTS=ON in VTK 6.3 which is not listed in installation instructions.

Lastly, I am not sure about this one but it also requires VTK and OpenCV to be build with QT.

@Aravind-Suresh
Copy link

Also, installation instruction says PCL v1.6+, but v1.6 doesn't have apps in it. The latest version v1.8 or the source has the required apps. This small change should also be added apart from the one mentioned above.

@krips89
Copy link
Owner

krips89 commented Mar 7, 2016

@IshankGulati Thanks for pointing this out. PCL build settings will determine its own dependencies (with all the details including versions) and should not be included here. To address your concern, I have mentioned about OpenNI in the dependency information of PCL, but not as an additional dependency.

Can you specifically point out, what fails without those options in VTK/OpenCV?

@Aravind-Suresh Thanks, I have updated the documentation.

@hanshammel1337
Copy link

I think there should be more improvment.
Under PCL 1.8 there is OpenNI mentioned. Obviously there are OpenNI and OpenNI2. Which version is correct?
And in the examples for 3D a kinect camera is used. Again: Which version? Kinect v1 (XBox 360) oder Kinect v2 (Xbox One)? It would be nice when this information were more precise.

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