Skip to content

Include the 39_autotune_roll_pitch_retune_results.param file#1424

Merged
amilcarlucas merged 1 commit intomasterfrom
fix_tuning_report
Mar 23, 2026
Merged

Include the 39_autotune_roll_pitch_retune_results.param file#1424
amilcarlucas merged 1 commit intomasterfrom
fix_tuning_report

Conversation

@amilcarlucas
Copy link
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings March 23, 2026 11:52
@amilcarlucas amilcarlucas merged commit 179e8d2 into master Mar 23, 2026
12 checks passed
@amilcarlucas amilcarlucas deleted the fix_tuning_report branch March 23, 2026 11:52
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the tuning report generation to reference the new 39_autotune_roll_pitch_retune_results.param file.

Changes:

  • Replaces the previously referenced roll/pitch autotune parameter file name with the retune variant in the tuning report inputs.

Comment on lines 952 to 956
"33_autotune_pitch_results.param",
"35_autotune_yaw_results.param",
"37_autotune_yawd_results.param",
"39_autotune_roll_pitch_results.param",
"39_autotune_roll_pitch_retune_results.param",
]
Copy link

Copilot AI Mar 23, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The PR title says “Include the 39_autotune_roll_pitch_retune_results.param file”, but the change replaces "39_autotune_roll_pitch_results.param" rather than adding the new file alongside it. If the intent is to support both outputs (existing results + retune results), keep "39_autotune_roll_pitch_results.param" in the list and add "39_autotune_roll_pitch_retune_results.param" as an additional entry; only replace it if the old filename is truly obsolete and should no longer be considered.

Copilot uses AI. Check for mistakes.
@github-actions
Copy link
Contributor

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
11803 11061 94% 89% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 30e69cf by action🐍

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.

2 participants