-
Notifications
You must be signed in to change notification settings - Fork 88
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
costmap_cspace: make Costmap3dLayerPlain and Costmap3dLayerOutput faster #562
Conversation
…p3dLayerOutput::generateUpdateMsg() faster
[16] FAILED on melodicTest failed
[16] FAILED on kineticTest failed
[16] FAILED on noeticTest failed
|
[17] PASSED on melodicAll tests passed
[17] PASSED on kineticAll tests passed
[17] PASSED on noeticAll tests passed
|
Codecov Report
@@ Coverage Diff @@
## master #562 +/- ##
==========================================
+ Coverage 90.78% 90.84% +0.06%
==========================================
Files 58 58
Lines 4428 4447 +19
==========================================
+ Hits 4020 4040 +20
+ Misses 408 407 -1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nothing major but I just left a few comments where we could further avoid some operations.
[18] PASSED on kineticAll tests passed
[18] PASSED on melodicAll tests passed
[18] PASSED on noeticAll tests passed
|
[19] PASSED on melodicAll tests passed
[19] PASSED on kineticAll tests passed
[19] PASSED on noeticAll tests passed
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think Costmap3dLayer
classes should avoid direct access to the underlying memory block.
CSpace3DMsg
should encapsulate them.
[20] PASSED on melodicAll tests passed
[20] PASSED on kineticAll tests passed
[20] FAILED on noeticTest failed
[20] PASSED on melodicAll tests passed
[20] PASSED on kineticAll tests passed
[20] PASSED on noeticAll tests passed
|
@at-wat I added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Costmap3dLayerPlain: As each map does not vary with its yaw angle, only first map is generated and copied to others.
Costmap3dLayerOutput: When an entire map is updated, all memories are copied at once. Otherwise, each raster is copied.