-
Notifications
You must be signed in to change notification settings - Fork 212
Conversation
Hello @tchaton! Thanks for updating this PR.
Comment last updated at 2021-07-16 17:06:58 UTC |
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.
Don't know much about pointclouds, but LGTM :)
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 😃
Codecov Report
@@ Coverage Diff @@
## master #600 +/- ##
==========================================
- Coverage 90.90% 89.91% -1.00%
==========================================
Files 132 140 +8
Lines 7795 8199 +404
==========================================
+ Hits 7086 7372 +286
- Misses 709 827 +118
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
What does this PR do?
Fixes # (issue)
Before submitting
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃