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

Noetic Updates Associated With reuleaux_moveit Pull Request 8 #4

Open
wants to merge 62 commits into
base: master
Choose a base branch
from

Conversation

MShields1986
Copy link

Updates for Noetic needed for usage with reuleaux_moveit PR8.

I have not tested this fork for map creation as I ran into issues building OpenRAVE and for my work I needed to use MoveIt. It would be good if someone could test the functionality of this branch for map creation using OpenRAVE.

jontromanab and others added 30 commits December 14, 2016 23:50
Without this `catkin build map_creator` fails. With this and running the following, I had all necessary dependency got resolved:

```
cd %YOUR_CATKIN_WS%
rosdep install -r -y --from-paths src --ignore-src
```
[Metapackage](http://www.ros.org/reps/rep-0142.html) is commonly added to a ROS-based package suite where multiple packages are in. It allows to specify a single package upon build, e.g. `catkin build reuleaux`, and it's always good for people to access the single point of entry for the package.
Add a config file for Travis CI, for automated testing. Please see more about what can be checked [doc](https://github.com/ros-industrial/industrial_ci/blob/master/README.rst). Testcases might be not yet added to the packages here but at minimum we can check if pkgs build, dependency clearance etc.

If this LGTY, any admins could enable Travis (at something like https://travis-ci.org/profile/ros-industrial-consortium) by a single click to get the checking started.
jontromanab and others added 29 commits September 27, 2019 12:40
@MShields1986
Copy link
Author

This is now up to date with the ROS-I master's latest commit also so we could PR to there also, once we are happy with 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.

6 participants