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
Update ecl to add ability to recover from bad magnetic yaw #14301
Conversation
Huge flash overflow on fmu-v2. Looks like it might be time for something a little more drastic... |
b1e878d
to
ac9eccc
Compare
ac9eccc
to
bb655f4
Compare
src/modules/logger/logger.h
Outdated
#ifdef ORB_USE_PUBLISHER_RULES | ||
static constexpr hrt_abstime TRY_SUBSCRIBE_INTERVAL {1000}; | ||
#else | ||
static constexpr hrt_abstime TRY_SUBSCRIBE_INTERVAL {1000 * 1000}; // interval in microseconds at which we try to subscribe to a topic | ||
#endif |
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.
Why do we need that?
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.
Removed for now. Might be less relevant after #14106. I'll readd this (or solve it differently) if needed in master.
Fix for CI clang-tidy failure PX4/PX4-ECL#775. |
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.
All good to me
Tested on PixRacer V4Indoor Flight Test Procedure Notes Log https://review.px4.io/plot_app?log=e5d5c88a-6cd0-4297-880c-54328d4c2018 Tested on CUAV nano V5Indoor Flight Test Procedure Notes Log https://review.px4.io/plot_app?log=442d5bf6-b8ac-48bc-b8dc-7befe8525b27 |
5c466e4
to
dbab114
Compare
63b24d5
to
30e4206
Compare
30e4206
to
82a55fd
Compare
Required to log additional data required for flight testing when ecl version updates to latest master containing PX4/PX4-ECL@4669aa6