Skip to content

Pr hybrid planning review#351

Merged
henningkayser merged 5 commits intomoveit:feature/hybrid_planningfrom
henningkayser:pr-hybrid_planning_review
Feb 1, 2021
Merged

Pr hybrid planning review#351
henningkayser merged 5 commits intomoveit:feature/hybrid_planningfrom
henningkayser:pr-hybrid_planning_review

Conversation

@henningkayser
Copy link
Copy Markdown
Member

@henningkayser henningkayser commented Jan 29, 2021

Fixes addressing mostly MoveIt best practices. @sjahr

@henningkayser henningkayser force-pushed the pr-hybrid_planning_review branch from de6b33a to bb9898d Compare January 29, 2021 14:20
@henningkayser henningkayser force-pushed the pr-hybrid_planning_review branch from bb9898d to 99ab5ac Compare January 29, 2021 14:24
@henningkayser henningkayser force-pushed the pr-hybrid_planning_review branch from 99ab5ac to 90b259c Compare January 29, 2021 14:39
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 29, 2021

Codecov Report

Merging #351 (90b259c) into feature/hybrid_planning (41afba5) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@                   Coverage Diff                    @@
##           feature/hybrid_planning     #351   +/-   ##
========================================================
  Coverage                    48.00%   48.00%           
========================================================
  Files                          157      157           
  Lines                        14843    14843           
========================================================
  Hits                          7124     7124           
  Misses                        7719     7719           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 41afba5...90b259c. Read the comment docs.

Copy link
Copy Markdown
Contributor

@sjahr sjahr left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@henningkayser henningkayser merged commit f33df4b into moveit:feature/hybrid_planning Feb 1, 2021
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Apr 16, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request May 4, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request May 4, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request May 9, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
henningkayser added a commit that referenced this pull request May 18, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
henningkayser added a commit that referenced this pull request May 18, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Jun 4, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Jun 4, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Jun 7, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Jun 7, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
henningkayser added a commit to henningkayser/moveit2 that referenced this pull request Jul 6, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Aug 21, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Sep 28, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Oct 19, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Oct 28, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
sjahr pushed a commit to sjahr/moveit2 that referenced this pull request Nov 18, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
AndyZe pushed a commit to AndyZe/moveit2 that referenced this pull request Nov 24, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
AndyZe pushed a commit to AndyZe/moveit2 that referenced this pull request Nov 29, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
AndyZe pushed a commit to AndyZe/moveit2 that referenced this pull request Nov 30, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
AndyZe pushed a commit to AndyZe/moveit2 that referenced this pull request Dec 3, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
AndyZe pushed a commit to AndyZe/moveit2 that referenced this pull request Dec 7, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
AndyZe pushed a commit to AndyZe/moveit2 that referenced this pull request Dec 7, 2021
* Export missing plugins
* Use std::chrono_literals
* Construct smart pointers with std::make_* instead of 'new'
* Fixup const-ref in function signatures, reduce copies
* Improve planning scene locking, robot state processing, controller logic
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.

2 participants