Skip to content

fix(ekf/mavlink): fix logging and mavlink stream of fusion-control-status#27882

Open
haumarco wants to merge 1 commit into
mainfrom
fix_fusion_control_status_reporting
Open

fix(ekf/mavlink): fix logging and mavlink stream of fusion-control-status#27882
haumarco wants to merge 1 commit into
mainfrom
fix_fusion_control_status_reporting

Conversation

@haumarco

Copy link
Copy Markdown
Contributor

Solved Problem & Solution

Currently, the estimator_fusion_control uORB topic wasn't being logged correctly. This change advertises it immediately.

Additionally, ESTIMATOR_SENSOR_FUSION_STATUS wasn't included in any low-bandwidth logging profile. Since this is a system-critical stream for the new fusion control logic over MAVLink, it has now been added.

Changelog Entry

For release notes:

Bugfix: fixed `estimator_fusion_control` logging and `ESTIMATOR_SENSOR_FUSION_STATUS` streaming

Test coverage

Tested in SITL.

…and low-bandwith streaming of ESTIMATOR_SENSOR_FUSION_STATUS
@haumarco haumarco requested a review from bresch July 13, 2026 12:30
@github-actions github-actions Bot added kind:bug Something is broken or behaving incorrectly. scope:estimation EKF, local position, attitude, wind, bias, or aiding logic. scope:mavlink MAVLink module, streams, commands, or protocol handling. labels Jul 13, 2026
@github-actions

Copy link
Copy Markdown
Contributor

🔎 FLASH Analysis

px4_fmu-v5x [Total VM Diff: 8 byte (0 %)]
    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +0.0%      +8  +0.0%      +8    .text
    +1.3%      +8  +1.3%      +8    EKF2::AdvertiseTopics()
  +0.0%     +55  [ = ]       0    .debug_abbrev
  +0.0%     +19  [ = ]       0    .debug_info
  -0.0%      -2  [ = ]       0    .debug_line
   -50.0%      -3  [ = ]       0    [Unmapped]
    +0.0%      +1  [ = ]       0    [section .debug_line]
  -0.1%      -8  [ = ]       0    [Unmapped]
  +0.0%     +72  +0.0%      +8    TOTAL

px4_fmu-v6x [Total VM Diff: 0 byte (0 %)]
    FILE SIZE        VM SIZE    
 --------------  -------------- 
  +0.0%     +55  [ = ]       0    .debug_abbrev
  +0.0%     +19  [ = ]       0    .debug_info
  +0.0%      +6  [ = ]       0    .debug_line
    +250%      +5  [ = ]       0    [Unmapped]
    +0.0%      +1  [ = ]       0    [section .debug_line]
  +0.0%     +80  [ = ]       0    TOTAL

Updated: 2026-07-13T12:38:01

@mbjd

mbjd commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

I remember seeing logs where various aid_src topics were not present even though we never disable verbose logging. Are these topics affected by a similar issue?

Here they are advertised. If when the logger initialises these branches do not run (EKF2_GPS_CTRL not fully on) can that completely disable logging them?

@bresch

bresch commented Jul 14, 2026

Copy link
Copy Markdown
Member

I remember seeing logs where various aid_src topics were not present even though we never disable verbose logging. Are these topics affected by a similar issue?

Here they are advertised. If when the logger initialises these branches do not run (EKF2_GPS_CTRL not fully on) can that completely disable logging them?

Yes, so if the _CTRL parameter is not set when the EKF starts, you'll not have those topics in the log.

@mbjd

mbjd commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Ah of course, it is because we still have some airframes that are not updated to the new runtime fusion control way, was confused

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug Something is broken or behaving incorrectly. scope:estimation EKF, local position, attitude, wind, bias, or aiding logic. scope:mavlink MAVLink module, streams, commands, or protocol handling.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants