Skip to content

Reset free PIDs if coefficients are changed via CLI or MSP#11431

Open
wimalopaan wants to merge 1 commit intoiNavFlight:masterfrom
wimalopaan:wm_pid_cli
Open

Reset free PIDs if coefficients are changed via CLI or MSP#11431
wimalopaan wants to merge 1 commit intoiNavFlight:masterfrom
wimalopaan:wm_pid_cli

Conversation

@wimalopaan
Copy link

Maybe solution for #11428

@github-actions
Copy link

Branch Targeting Suggestion

You've targeted the master branch with this PR. Please consider if a version branch might be more appropriate:

  • maintenance-9.x - If your change is backward-compatible and won't create compatibility issues between INAV firmware and Configurator 9.x versions. This will allow your PR to be included in the next 9.x release.

  • maintenance-10.x - If your change introduces compatibility requirements between firmware and configurator that would break 9.x compatibility. This is for PRs which will be included in INAV 10.x

If master is the correct target for this change, no action is needed.


This is an automated suggestion to help route contributions to the appropriate branch.

@sensei-hacker
Copy link
Member

Thanks for this.
You said "maybe". Does it seem to work?

@wimalopaan
Copy link
Author

Thanks for this. You said "maybe". Does it seem to work?

Tested CLI and INAV-Configurator.
For me it works, but be aware that the PID is fully reset, that is the actual state after this will be 0. But I think this is ok.

@github-actions
Copy link

Test firmware build ready — commit ec61575

Download firmware for PR #11431

225 targets built. Find your board's .hex file by name on that page (e.g. MATEKF405SE.hex). Files are individually downloadable — no GitHub login required.

Development build for testing only. Use Full Chip Erase when flashing.

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