Skip to content

Add more milestones and detail to ROS2 roadmap#17

Merged
davetcoleman merged 3 commits intomasterfrom
readme-milestones
Mar 4, 2019
Merged

Add more milestones and detail to ROS2 roadmap#17
davetcoleman merged 3 commits intomasterfrom
readme-milestones

Conversation

@davetcoleman
Copy link
Copy Markdown
Member

No description provided.

- [ ] Setup instructions
- [ ] Install instructions
- [ ] Ubuntu 18.04
- [ ] OS X 10.14
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vmayoral my understanding is that you don't have plans to support OSX currently, right?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davetcoleman we discussed this internally and we're going to try to get OS X support as well. That's the initial effort at least.

- [ ] Install instructions for Ubuntu 18.04 in README.md
- [ ] Upgrade continuous integration for ROS 2.0
- [ ] Convert moveit packages to ROS 2.0
- [x] ~~Include moveit_msgs in HRIM (**@ibaiape**)~~
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@vmayoral what is HRIM?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@davetcoleman https://github.com/AcutronicRobotics/HRIM, discarded now as part of our contributions due to community's input. Refer to ros2/ros2#607 (comment)

- [ ] Update tutorials for MoveIt2
- [ ] Create tutorial on using ros1/ros2 bridge to support ros1 hardware drivers
- [ ] Move install instructions to moveit.ros.org
- [ ] Major refactoring and divergence from moveit2
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rhaschke some of the breaking changes i'm hoping to do for moveit2, to be discussed

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Apart from moving to ROS2 structures instead of plugin interfaces - and this is not even listed here - I do not see a single item in this list I would hesitate to discuss for moveit-for-ros1's master branch too.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok, well let's get past the ros1 release and master branch split and discuss that there!

@vmayoral
Copy link
Copy Markdown
Contributor

Made some changes on top of this branch @ https://github.com/AcutronicRobotics/moveit2/tree/readme-milestones (don't have permissions to push here).

@vmayoral
Copy link
Copy Markdown
Contributor

@davetcoleman please cherry-pick AcutronicRobotics@8a2e766

@davetcoleman
Copy link
Copy Markdown
Member Author

@vmayoral Its difficult to discuss/debate your requested changes via your request to cherry-pick, some of them revert the point of my pull request so could you just request those changes here? you can use the github 'suggest' feature if you'd like. or you can make a follow-up PR after this one is merged in if you want to add more.

- [x] Upgrade continuous integration for ROS 2.0
- [x] Refactor/cleanup folder hierarchy
- [ ] Install instructions for Ubuntu 18.04 in README.md
- [ ] Upgrade continuous integration for ROS 2.0
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add item removed, not sure where though

Suggested change
- [ ] Upgrade continuous integration for ROS 2.0
- [ ] Upgrade continuous integration for ROS 2.0
- [x] Update/setup infrastructure for development
- [x] Delete metapackages
- [x] Upgrade continuous integration for ROS 2.0
- [x] Refactor/cleanup folder hierarchy

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I expanded these with more details, below:

  • Delete excesses packages that are left over from rosbuild stacks: moveit_runtime, moveit_plugins, moveit_ros
  • Rename non-package folders:
    • rename moveit_planners to planners
    • rename moveit_plugins to controller_interfaces
  • Restructure folder layout of moveit repo:
    • flatten moveit_ros folder to root of repo
    • rename all moveit_ros folders with moveit_ros prefix

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That looks good @davetcoleman

Co-Authored-By: davetcoleman <dave@picknik.ai>
@davetcoleman
Copy link
Copy Markdown
Member Author

Need approval so I can merge

@mlautman

mlautman
mlautman previously approved these changes Mar 4, 2019
Co-Authored-By: davetcoleman <dave@picknik.ai>
@davetcoleman davetcoleman merged commit c9c30a2 into master Mar 4, 2019
@davetcoleman davetcoleman deleted the readme-milestones branch March 4, 2019 23:25
MikeWrock pushed a commit to MikeWrock/moveit2 that referenced this pull request Aug 15, 2022
* moveit commander

* removing ref to pr2 and minor cleanup to moveit_commander tutorial

* changing bold to code
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.

4 participants