-
Notifications
You must be signed in to change notification settings - Fork 2
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #31 +/- ##
========================================
Coverage 75.34% 75.34%
========================================
Files 9 9
Lines 361 361
Branches 40 40
========================================
Hits 272 272
Misses 82 82
Partials 7 7
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.
Good job, I have tested a few, so let's hope they all work 🤞
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.
Very very nice! Your really put in some effort :O
A few comments on axis:
For the temprature plot, the axis are automatic instead of relative. Is this on purpose? Maybe nice indeed to see temprature over entire time
PID_update plot has absolute y-axis. Perhaps relative is better here?
Controller_output plots points instead of lines.
Closes PM-335
Description
This reintroduces the central folder for rqt_multiplot configurations. Making this available for everyone should help in getting the necessary plots fast.
Some general comments:
Available plots:
Changes