Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[uavcan] ESC temperature and motor temperature #3088

Merged
merged 2 commits into from
Sep 21, 2023
Merged

[uavcan] ESC temperature and motor temperature #3088

merged 2 commits into from
Sep 21, 2023

Conversation

dewagter
Copy link
Member

@dewagter dewagter commented Sep 20, 2023 •
edited
Loading

@dewagter
Copy link
Member Author

requires: paparazzi/pprzlink#173

@dewagter dewagter marked this pull request as draft September 20, 2023 17:11
dewagter added a commit to tudelft/paparazzi that referenced this pull request Sep 20, 2023
@dewagter
UAVCAN Temp2 ----> paparazzi#3088 and RPM ----> paparazzi#3089
9ecd1b5
@dewagter dewagter marked this pull request as ready for review September 21, 2023 09:32
@dewagter dewagter mentioned this pull request Sep 21, 2023
Closed
64 tasks
fvantienen
fvantienen previously approved these changes Sep 21, 2023
View reviewed changes
dewagter added a commit to tudelft/paparazzi that referenced this pull request Sep 21, 2023
@dewagter
UAVCAN Temp2 ----> paparazzi#3088 and RPM ----> paparazzi#3089
a5eeb37
@dewagter
Copy link
Member Author

221c0c2 -> make test

Files=1, Tests=27, 1757 wallclock secs ( 0.06 usr  0.02 sys + 1634.39 cusr 376.13 csys = 2010.60 CPU)
Result: PASS

All tests successful.
Files=1, Tests=169, 200 wallclock secs ( 0.06 usr  0.09 sys + 174.19 cusr 22.32 csys = 196.66 CPU)
Result: PASS

@dewagter dewagter merged commit ca1c1db into master Sep 21, 2023
@dewagter dewagter deleted the esc_and_motor_temps branch September 21, 2023 17:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@gautierhattenberger gautierhattenberger gautierhattenberger left review comments

@fvantienen fvantienen fvantienen left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@dewagter @gautierhattenberger @fvantienen