Skip to content
This repository was archived by the owner on Nov 13, 2017. It is now read-only.

Send Octomap diff instead of whole tree - #255

Closed
TheBrewCrew wants to merge 2 commits into
moveit:indigo-develfrom
TheBrewCrew:cbrew_octomap_diff
Closed

Send Octomap diff instead of whole tree#255
TheBrewCrew wants to merge 2 commits into
moveit:indigo-develfrom
TheBrewCrew:cbrew_octomap_diff

Conversation

@TheBrewCrew

Copy link
Copy Markdown

This enables Moveit to pass only the diff of the Octomap in a scene diff message when change detection is enabled on the Octree. This pull request depends on OctoMap/octomap#85 to compile. DO NOT MERGE until the Octomap pull request is accepted. moveit/moveit_ros#602 is the corresponding pull request in Moveit ROS for enabling change detection on the Octomap.

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.

This should probably be sizeof(float) instead.

@dornhege

Copy link
Copy Markdown
Contributor

+1 for this.

However, this cannot be accepted as OctoMap/octomap#85 will not be released for indigo due to ABI incompatibility. I'll prepare a pull request based on this that only requires OctoMap/octomap#103, which should be releasable.

@dornhege

Copy link
Copy Markdown
Contributor

Closing this in favor of #274.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants