CentroEPiaggio / pacman-DR53

This repository contains the necessary tools to run the demo contemplated in DR 5.3 of the PaCMan Project.
GNU General Public License v2.0
0 stars 0 forks source link

Object Pose Estimation #19

Closed carlosjoserg closed 9 years ago

carlosjoserg commented 9 years ago

The current state of the object pose estimation recognizes point cloud patches and not an object mesh. Hence, the reference coordinate frame for the object is not unique. Things to do:

This is a pre-requisite of #5 since it is needed to calibrate the star tracker with objects.

carlosjoserg commented 9 years ago

@Tabjones I think we can tick all items in the list above since the procedure is established, and object pose estimation is working, right? At least for few objects it is there.

Tabjones commented 9 years ago

@carlosjoserg Yes currently all steps above are accomplished for a few objects (5). I was waiting to have all the objects, but i can tick them and leave the issue open until all objects are completed.

carlosjoserg commented 9 years ago

:+1:

carlosjoserg commented 9 years ago

Hey @Tabjones , I see that the pose estimation depends on 1.7.2 of PCL

And I can't find it as a standard package in synaptics/apt-get, only 1.7.1 is there, or perhaps I'm missing another repository. Can you confirm that requirement?

Another thing, the install of this should be automated. It'd be good if the whole pacman-DR53 compiles with a single catkin_make command. My suggestion (to keep your repos clean): try convertin the perception folder as a meta-package, and add post-build and dependencies CMake commands.

Tabjones commented 9 years ago

yo @carlosjoserg, yes it needs pcl 1.7.2. i'm using some functionalities that are not there in 1.7.1, and yes ubuntu 14.04 doesn't have it. Recommended way to solve this is to clone pcl github repo, check out to 1.7.2 (git checkout pcl-1.7.2), configure and make install it (it will go in /usr/local/lib). cmakelists in perception packages are configured to give priority to the local installed pcl instead of the system one. Cmake will complain a bit, it will tell you that you are "hiding" some libraries in /usr/lib in favor of some in /usr/local/lib, (thanks a lot, that was what i was trying to do!) but everything will work fine. Yes i should probably mention this in the various readmes, and will do it.

As for the second thing, i'm not sure how i can tell catkin to make install a 3rd party library, before it actually tries to build perception packages, but i will look into it. Probably what you are suggesting is the correct/recommended way to do this.

carlosjoserg commented 9 years ago

Recommended way to solve this...

Ok, no standard synaptic/apt-get then, so I would suggest to automate the process of getting it. Two options I can think of:

As for the second thing, i'm not sure how i can tell catkin to make install a 3rd party library, before it actually tries to build perception packages...

You can use standard CMake for that as well, even within catkin, for instance a POST-BUILD custom command like this one might be a good option, but in your case as a PRE-BUILD. Here you have the full specs.

carlosjoserg commented 9 years ago

@Tabjones Should we close this?

Tabjones commented 9 years ago

Yes !