Skip to content

feat(simulation): Disable sensor calibration in HIL#26942

Open
ttechnick wants to merge 1 commit intomainfrom
pr-disable-cal-sih
Open

feat(simulation): Disable sensor calibration in HIL#26942
ttechnick wants to merge 1 commit intomainfrom
pr-disable-cal-sih

Conversation

@ttechnick
Copy link
Copy Markdown
Member

@ttechnick ttechnick commented Apr 1, 2026

Solved Problem

In SIH you can currenly calibrate the sensors, which will have no effect, but mess up the real vehicle when switching back to the airframe.

Solution

This pull request introduces logic to prevent sensor calibration commands from being accepted when the system is running in SIHmode. This is important because simulated sensors in SIH mode cannot be calibrated, so the system now explicitly denies such commands and provides clear feedback to the user.

Changelog Entry

For release notes:

Feature: Prevent sensor calibration when in SIH

Testing

Tested in:

  • Hardware SIH (cal disabled)
  • SITL SIH (cal enabled)
  • Hardware (cal enabled)

@ttechnick ttechnick self-assigned this Apr 1, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 1, 2026

🔎 FLASH Analysis

px4_fmu-v5x [Total VM Diff: 208 byte (0.01 %)]
    FILE SIZE        VM SIZE    
--------------  -------------- 
+0.0%    +208  +0.0%    +208    .text
  +3.4%    +120  +3.4%    +120    Commander::handle_command()
  +0.0%     +56  +0.0%     +56    [section .text]
  +0.0%     +32  +0.0%     +32    g_cromfs_image
+0.0%     +55  [ = ]       0    .debug_abbrev
+0.0%    +302  [ = ]       0    .debug_info
+0.0%    +147  [ = ]       0    .debug_line
  [NEW]      +6  [ = ]       0    [Unmapped]
  +0.0%    +141  [ = ]       0    [section .debug_line]
+0.0%    +270  [ = ]       0    .debug_loclists
-0.0%      -2  [ = ]       0    .debug_rnglists
-2.1%    -208  [ = ]       0    [Unmapped]
+0.0%    +772  +0.0%    +208    TOTAL

px4_fmu-v6x [Total VM Diff: 208 byte (0.01 %)]
    FILE SIZE        VM SIZE    
--------------  -------------- 
+0.0%    +208  +0.0%    +208    .text
  +3.4%    +120  +3.4%    +120    Commander::handle_command()
  +0.0%     +56  +0.0%     +56    [section .text]
  +0.0%     +28  +0.0%     +28    g_cromfs_image
  +4.8%      +4  +4.8%      +4    FlightTask
+0.0%     +55  [ = ]       0    .debug_abbrev
+0.0%    +302  [ = ]       0    .debug_info
+0.0%    +139  [ = ]       0    .debug_line
 -40.0%      -2  [ = ]       0    [Unmapped]
  +0.0%    +141  [ = ]       0    [section .debug_line]
+0.0%    +270  [ = ]       0    .debug_loclists
+0.0%      +2  [ = ]       0    .debug_rnglists
-4.2%    -208  [ = ]       0    [Unmapped]
+0.0%    +768  +0.0%    +208    TOTAL

Updated: 2026-04-01T15:46:40

@ttechnick ttechnick marked this pull request as ready for review April 1, 2026 16:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant