Hrishikesh Gohain
Hrishikesh Gohain
Hi @peterbarker, I'm new to ArduPilot and would like to take up this issue as my first contribution. I've read through the contribution guidelines and explored the codebase. **My take...
I did the changes as mentioned @Ryanf55 and build and tested the code according to the docs. Could you please review when you have time? I'm happy to make any...
Not ready to merge — contains breaking changes. (Picked commit 1af26163ed1b018ca2ef3b93fe78469ba35269cc from mavlink/mavlink as suggested on Discord.) Needs a PR in ArduPilot to fix these breaking changes (WIP).
Hi @peterbarker , I have cleaned up my branch by removing the merge commits and rebased it onto the latest master as requested. I am now working on applying the...
@peterbarker I've squashed the commits into one and pushed the updated version with the requested changes. Let me know if anything else is needed.
> #31138 should fix the need for the extra set, but there is some more cleanup to do. Are you interested in further work on this? Sure! I’ll start with...
I simplified the getters and made them const ```cpp // get the active main loop rate uint16_t get_loop_rate_hz(void) const { return _active_loop_rate_hz; } // get the time-allowed-per-loop in microseconds uint32_t...
@tpwrules I implemented the suggested changes and also included some changes previously mentioned by @peterbarker. I’ve opened PR #31165 — I’d appreciate a review and any further suggestions. Happy to...