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 minor cleanup #13051

Merged
merged 3 commits into from
Oct 28, 2019
Merged

uavcan minor cleanup #13051

merged 3 commits into from
Oct 28, 2019

Conversation

dagar
Copy link
Member

@dagar dagar commented Sep 29, 2019 •
edited
Loading

  • move most orb to uORB::Publication and uORB::Subscription
  • update legacy message handling (warn to PX4_INFO, PX4_WARN, PX4_ERR)
  • add perf counters
  • sensors/mag support newer uavcan::equipment::ahrs::MagneticFieldStrength2 message
  • sensors/gps support uavcan::equipment::gnss::Auxiliary for hdop and vdop
  • sensors delete obsolete ioctl and read methods
  • use PublicationMulti for actuator_outputs and esc_reports (to coexist with other output modules)
  • add GNSS parameter metadata (parameters_injected.xml)

@weekly-digest weekly-digest bot mentioned this pull request Oct 6, 2019
Closed
@bkueng bkueng mentioned this pull request Oct 23, 2019
Merged
2 tasks
pavel-kirienko
pavel-kirienko previously approved these changes Oct 26, 2019
View reviewed changes
@dagar dagar force-pushed the pr-uavcan_cleanup_minor branch from c7396c3 to 5981207 Compare October 28, 2019 22:32
@dagar dagar merged commit 375fc4a into master Oct 28, 2019
@dagar dagar deleted the pr-uavcan_cleanup_minor branch October 28, 2019 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pavel-kirienko pavel-kirienko pavel-kirienko left review comments

@bkueng bkueng Awaiting requested review from bkueng

Assignees
No one assigned
Labels
Admin: Cleanup (housekeeping) 🧹 Admin: Enhancement (improvement) 💡 UAVCAN
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dagar @pavel-kirienko @TSC21