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

Error: template argument required for 'class AABB' #186

Closed
ytzhao opened this issue Nov 8, 2016 · 2 comments
Closed

Error: template argument required for 'class AABB' #186

ytzhao opened this issue Nov 8, 2016 · 2 comments

Comments

@ytzhao
Copy link

ytzhao commented Nov 8, 2016

Hi,
I'm using FCL for manipulator collision check, this library is very new to me. I followed the instructions to install this library on Ubuntu16.04 without ROS. I used 'general_test.cpp' for testing in Codeblocks with following build message:
'error: template argument required for 'class AABB''.

I've already included the library path and .h files in the project. I had several tries and found that if I commented <fcl/shape/geometric_shapes.h>, <fcl/shape/geometric_shapes_utility.h>, <fcl/narrowphase/narrowphase.h> these three files, there's no compiling error.

Once I included either of these three header files, the compiling error appears. I traced down into these files and found that there are two AABB.h files in the library. One is <fcl/BV/AABB.h> and another is <fcl/math/bv/AABB.h>. The later file is written with 'template' while the former one is not. It seems that if the former file is included, there's always the same error. So what's the different use between these two files? Is the one with template the newest version?

Anyone have ideas for solving this problem?

Thanks! :-)

@jslee02
Copy link
Member

jslee02 commented Nov 10, 2016

general_test.cpp hasn't been updated for the new API (FCL 0.6). It was an oversight. I'll create a PR that updates the test.

The templated one is the newest version. I guess the non-templated one is the old version that's installed in the system directory.

@ytzhao
Copy link
Author

ytzhao commented Nov 11, 2016

Thank you! Anyway, I reinstalled this library with more stable version 0.5, OpenRAVE and OMPL depend on the former version.

@ytzhao ytzhao closed this as completed Nov 11, 2016
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

2 participants