iNavFlight / inav

INAV: Navigation-enabled flight control software
https://inavflight.github.io
GNU General Public License v3.0
3.08k stars 1.46k forks source link

Variables in the configurator are mixed up. #10297

Open and-sh opened 1 month ago

and-sh commented 1 month ago

when platform = multirotor pid_type= PIFF

Current Behavior

Configurator and blackbox show nav_fw variables FC uses nav_mc variables

fig1. - all velosity z variables set to zero in Inav configurator.

nav1

fig2. - althold PID works

nav2

fig3. - all **nav_mc_velz*** variables set to zero trough CLI. althold PID does not work.

nav3

Steps to Reproduce

set platform = multirotor pid_type= PIFF

Try to tune navigation.

Expected behavior

Show and change the variables actually used.

Suggested solution(s)

At least write warning in configurator when such combination are used.

Additional context


Used INAV 7.1.2 MATEKF722

stronnag commented 1 month ago

Configurator issues belong in the Configuration repository BlackboxExplorer issues belong in the BlackboxExplorer repository

and-sh commented 1 month ago

It is rather misunderstanding of control logic than configurator or blackbox issue. I guess osd have the same error. Not looked yet.