-
Notifications
You must be signed in to change notification settings - Fork 13.3k
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
Logger: add mode to enable/disable logging via AUX1 RC channel #12100
Conversation
36c10cc
to
4de1697
Compare
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.
Awesome!
Thank you Beat this is great news. I will test this on coming Saturday. I am not that familiar with the selection of the test versions. I see this is not in the master. Is my assumption correct, if I build it as "make px4_fmu-v5_default" then I get the code with your changes? Of course first a clone from github PX4. |
It is now on master. If you select the latest development build in QGC on Saturday, you should have it. |
I promised to test it this saturday. |
Beat, I did a few flights during the last week testing this feature. All worked ok. |
@jstrebel I don't think we would add features into a patch release. Patch releases are only for bugfixes. |
This was asked for in https://discuss.px4.io/t/start-and-stop-px4-logging-via-a-rc-channel-feature-request/10302.