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

Catkinized iai_maps and adapted it to the new version of KnowRob #6

Merged
merged 8 commits into from
Aug 12, 2014

Conversation

mtenorth
Copy link
Contributor

@mtenorth mtenorth commented Aug 1, 2014

As discussed in #4, I have split iai_maps into three packages such that each of them contains only one kind of map. All three packages have been catkinized. I have left all .launch files in iai_maps so that you don't have to adapt launch files that may include them.

The changes in detail:

  • iai_maps: 2D maps for localization -> formerly subfolders 'maps', 'launch'
  • iai_kitchen_defs: URDF model of the kitchen -> formerly subfolders 'defs', 'room', 'Media'
  • iai_semantic_map: Semantic OWL map of the kitchen -> formerly subfolders 'prolog', 'owl'

One issue to be considered: The semantic map has been converted to the format used in the new, catkinized KnowRob. It's mostly the same, only some package names of dependencies have changed, and you can now load OWL files using package://-style URLs. However, this means that after merging this pull request, all of us will have to switch to the new KnowRob version.

I'd therefore propose to first test if everything works for those of you who use KnowRob and iai_maps (probably mostly Jan) using this fork.

@mtenorth
Copy link
Contributor Author

Merged pull request for catkinized and refactored version of iai_maps. In case you continue to use the rosbuild version of knowrob, you will need to checkout the 'rosbuild' tag of this repository using 'git checkout rosbuild'.

mtenorth added a commit that referenced this pull request Aug 12, 2014
Catkinized iai_maps and adapted it to the new version of KnowRob
@mtenorth mtenorth merged commit 2f29098 into code-iai:master Aug 12, 2014
@gaya-
Copy link
Member

gaya- commented Sep 2, 2014

Ok, so for those changing to the catkinized version of iai_maps:

  1. check out the knowrob repo into your catkin workspace
  2. switch to the indigo-devel branch
  3. $ sudo apt-get install ros-DISTRO-rosjava
  4. update your rosdep to make sure it knows about swi-prolog-java ($ rosdep update)
  5. $ sudo apt-get install swi-prolog-java
  6. $ catkin_make in your catkin workspace
  7. if you get errors about gradle being silly with symbolic links, it is because the link is broken or doesn't exist at the build time, try to repair it.

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

Successfully merging this pull request may close these issues.

None yet

2 participants