-
Notifications
You must be signed in to change notification settings - Fork 13.4k
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
MAVlink forwarding #7670
MAVlink forwarding #7670
Conversation
nice, i'll have a look |
I added changes that allow relaying when it's from the same systemID but a different component. This is required to relay statustext messages and have them show up in the GCS |
Tested, works |
src/modules/mavlink/mavlink_main.cpp
Outdated
@@ -1999,6 +2006,9 @@ Mavlink::task_main(int argc, char *argv[]) | |||
configure_stream("NAMED_VALUE_FLOAT", 1.0f); | |||
configure_stream("VFR_HUD", 4.0f); | |||
configure_stream("WIND_COV", 1.0f); | |||
// Image captured runs at full rate independent of | |||
// any limit | |||
configure_stream("CAMERA_IMAGE_CAPTURED", 1.0f); |
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.
I need to remove this - this doesn't work without further changes.
src/modules/mavlink/mavlink_main.cpp
Outdated
//camera trigger is rate limited at the source, do not limit here | ||
configure_stream("CAMERA_TRIGGER", 500.0f); | ||
configure_stream("CAMERA_IMAGE_CAPTURED", 5.0f); | ||
// Camera trigger runs at full rate independent of |
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.
I need to revert this - this doesn't work without further changes.
src/modules/mavlink/mavlink_main.cpp
Outdated
@@ -2084,8 +2097,12 @@ Mavlink::task_main(int argc, char *argv[]) | |||
configure_stream("NAMED_VALUE_FLOAT", 50.0f); | |||
configure_stream("VFR_HUD", 20.0f); | |||
configure_stream("WIND_COV", 10.0f); | |||
configure_stream("CAMERA_TRIGGER", 500.0f); | |||
configure_stream("CAMERA_IMAGE_CAPTURED", 5.0f); | |||
// Camera trigger runs at full rate independent of |
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.
I need to revert this - this doesn't work without further changes.
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.
Needs more changes as I introduced unnecessary churn.
@LorenzMeier can you revert your camera changes so we can get this in? |
Good to merge. I also verified memory footprint and made sure UAVCAN setups should still be fine. |
Jenkins test this please. |
This removes the confusing ugly magic number of 233 introduced as part of #7670. Also, the convoluted if is cleaned up using 3 separate bools with some comments to explain what's going on. This should not change anything function-wise except that the flight controller could now potentially also use system ID 233 and not break forwarding.
This removes the confusing ugly magic number of 233 introduced as part of #7670. Also, the convoluted if is cleaned up using 3 separate bools with some comments to explain what's going on. This should not change anything function-wise except that the flight controller could now potentially also use system ID 233 and not break forwarding.
This removes the confusing ugly magic number of 233 introduced as part of #7670. Also, the convoluted if is cleaned up using 3 separate bools with some comments to explain what's going on. This should not change anything function-wise except that the flight controller could now potentially also use system ID 233 and not break forwarding.
This removes the confusing ugly magic number of 233 introduced as part of #7670. Also, the convoluted if is cleaned up using 3 separate bools with some comments to explain what's going on. This should not change anything function-wise except that the flight controller could now potentially also use system ID 233 and not break forwarding.
@sanderux This hopefully fixes the issues you were facing with forwarding. Could you cross-test and fix where required?