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
Rely on AP_Vehicle to handle main loop() #13399
Conversation
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.
Lgtm
125917e
to
ac7b91e
Compare
Parameters may not be available in the constructor, and that's what we need to be clamping.
So the loop rate gets clamped before we memoise it and the loop period in AP_Scheduler
ac7b91e
to
ad8dec6
Compare
I was concerned that constructor ordering might cause parameters not to be available in the scheduler at boot. Turns out that it was no worse than master, which clamped the value before we loaded parameters... Adding patches on top to fix that. |
I flashed this onto a Minipix - Rover, Plane and Copter. Seems to work as intended. |
I just flew this one a Solo. As discussed on devcall, merging. |
Tested on AntennaTracker on CubeBlack.