-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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 uorb top documentation re: Mavlink Console #2410
base: v1.13
Are you sure you want to change the base?
Conversation
Edit the uorb top documentation to reflect that the command does not work reliably in the Mavlink Console
Hey @dagar is this a bug or expected behavior? |
I guess whether it's a bug or expected behavior, it still seems like it'd be useful to note that it does not work over the Mavlink console. |
@msberk I agree this is worth having a note and really appreciate you taking the time to add one :-). But whether or not it is a bug or not defines what the comment looks like. For example, I like to add a bug number so that this kind of note can easily be checked and removed when the bug is fixed. The conversation with alex just says it isn't working in this case, but I can't find an open issue or bug report I can link against. I did try this locally and it works for me over USB to a CUAV Pixhawk 6x with latest main. Can you provide more detail. If not we can either create an issue for tracking and link it - or perhaps water-down this to "anecdotally". |
What I don't want to do is have an instruction that means people have to run into this problem forever. MAVLink console is too useful for this not to work. |
@hamishwillee sorry if I seemed short in the reply - I misinterpreted the comment to imply that if it was expected it may not need to be edited. The bug report linking makes sense for being able to track and remove notes such as this. Glad to hear the issue doesn't seem to exist in |
That would be great. If we can get a specific reproducible case then we can create an issue and move forward. |
Hi @msberk Did you manage to get any further testing done on this? |
Have not yet - it's on my todo list but want to make sure you know I haven't forgotten! Will update when I get to it. |
THanks, and no worries. I'm on other things too :-) |
Edit the
uorb top
documentation to reflect that the command does not work reliably in the Mavlink Console. This is in response to @AlexKlimaj's comment on Discord recently after I was unable to runuorb top
over the Mavlink Console on a Pixhawk 6X running PX4 1.13.2.If this is still applicable to main then this change should likely be applied there, too.