-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Inav 8.0 RC3 Airplane bumpiness #10536
Comments
Try adjusting the following settings which may be affected by changes in 8.0: set See also #9471. |
Okay. I'll try to check in a week, weather permitting. |
It would be nice to add this description when switching from previous versions to the new one. |
Are these universal settings or do have to select the settings for each aircraft yourself? |
Yes, it seems to have been missed and should be added, probably to the Important Changes section @mmosca.
Hard to say. I've used the same settings as yourself for PID on a conventional plane and flying wing and not really noticed any pitch instability. I also increased |
Before we merge the changes in #10541 I would really like to see a blackbox log of that flight. I am hesitant to change the altitude PIDs too low as usually no one tunes them afterwards and too soft values can result in issues with WP missions or Autoland. |
The current altitude PIDS are for altitude control based on position though whereas the altitude control in 8.0 is now based on vertical velocity instead. It uses the same PID controller but with velocity error rather than position error. The basic PID tuning factors: inav/src/main/navigation/navigation.c Line 4961 in 4389286
were corrected to take account of this by changing them from 10 to 100. This is based on the technical difference in required values that falls out from switching from position to velocity and also on what seemed to work from testing. The basic tuning factors weren't fine tuned any further. So it's likely the current default PIDs could be better tuned compared to previously. Reducing P fits in with what @Jetrell found from testing and others so it shouldn't be too much of an issue. It would be useful if @kasatka60 could test with lower P values to see if it helps fix the issue in this case confirming this is what's wrong. Is the Dolphin a bit pitch sensitive, seems to be the plane that others have had issues with ? |
Inav 7.1.2 |
Alright guys in this case lets see the blackbox and @kasatka60 ideally you fly with current values one more time, then with the changed values as breadoven said. Just to 100% exclude any PIDFF tuning issue. Its the fast oscillation in your clip above that throws be off a bit. Especially as no one else noted that issue so far. If the new defaults for vertical velocity pids work well and don't lack of precision (it would be bad if we have a 5m altitude error that takes 1km to correct), I am fine. I wish I had known about the necessary default changes to do more focus testing beforehand. |
@kasatka60 Just inquiring. Did this value
@b14ckyy Lets not loose track of the fact this controller operates on a different method.. No need for the sarcasm. Its disrespectful towards @breadoven's work. |
Sorry it was not meant to be sarcastic. Just somewhat exaggerated. I understand that the method needs new values. I just want to mess these defaults up based on a plane that is completely out of tune elsewhere and make it worse for everyone else. I am all in for good values but they should be flown and tested on at least well autotuned planes. Then we can still tweak them somewhat conservative from there. But they should not cover up a completely messed setup. |
Is that like a pat on the back, then a slap in the face towards Breadoven and myself. In saying we know nothing about testing and tuning aircraft ? And as I said yesterday, among other things.. This controller is more accurate, but this also makes it more sensitive.. Which will also make slight altitude oscillations more obvious on some airframes, that aren't tuned or setup as well. Not withstanding the dynamic vibration clipping, with its side effect, that was added around the same time as the velocity controllers original commit. That know-one except @ultrazar @breadoven had noticed as a significant issue. #10308 |
Values obtained as a result of autotune. |
Thanks @kasatka60 Being that autotune can set such a drastic difference in feedforward and likely rates too, on the same model, is interesting to see. That would mean any small oscillations from over tune of the |
Mate you are interpreting too much into here. I am talking about the Setup of the poster that apparently has a completely out of tune setup. Such a case should not be the reference. So my question here is, if not answered already. Have all your tests in the last 12 months been stable and precise on the current defaults or did you have to reduce them to values close to here: #10541 ? The main reason why I wanna know that is, because for years we set The bigger problem is also, that everyone who Upgrades from 7.1 with a diff, will keep the values from the old preset, even if you change the firmware defaults. As a result, imho this parameter should have been renamed to PS: I skipped through the video posted above. I can't tell for sure what FM was active most of the time but I see possibly a lot of flying in Angle mode and a swinging pitch axis. So after pitch changes the plane waves up and down a few times and then settles. This is an indicator of a overtuned FF. Can only say for sure with the logs. I had that phenomenon myself especially on my Dart250 since a speed stall due to angle of attack from strong pitch authority can sometimes be unnoticed on a swept forward. So the control output is high but the actual resulting pitch rate stays low in that stall situation. This tunes rates too low and FF too high resulting in what we see. |
Fair enough.. I just figured you would have read about it all. There is plenty of conversation between breadoven and myself in that PR.. But you must not have. No problems then 👍
Many things changed over the year, as we become more aware of possible causes. That was why its merge was held off for so long. |
@mmosca OK mate I think based on Jetrells report and based on the fact that I also had no issues in my tests with the Configurator preset values of 25 and 35 (the new default would be in the middle) on 5 different planes, I think we can merge the default value change in #10541 I will also remove that value from the presets in configurator. We should add a Upgrade note to the release notes that in case of Oscillation issues, the pilot should reduce that value. Alternatively we rename it as mentioned above? |
You're right that adding new And don't forget that if you add I don't think this is too big an issue so long as people are aware the PID settings might need adjusting if copied across from a previous version. |
Fair enough. In the end only No-Tail Planes are 5 points above firmware default after the change and I guess in most cases it won't even be a problem if the plane is overall tuned well. Planes with tail are 5 points below FW default. I edit the preset, add a note to the release notes and we are good to go. And for the @kasatka60: When you fly your dolphin again to create a blackbox log, reduce the FF value for pitch to 160 (just a guess on my end based on past experiences) for the second flight and see if its better. We can then compare the logs with your current values and with reduced FF to see if the tune gets better. |
I spent most of my time flying on cruis mode. |
Good to know, so the bumpiness shown above was just occasionally then? Probably after a "harsh" correction? |
I haven't adjusted anything yet. I transferred the diff settings. |
I ordered a memory card. I'll try flying in rain and snow and write down the logs. Just don't swear. |
OK so reducing P to 30 doesn't fix the issue. Also I assume the 3rd test was using a I did notice though that you have fw_d_pitch = 5 whereas I have it always set to the default of 0. Don't know if this makes much of a difference. You'd think it might help damp the oscillation but maybe not, D seems a little unpredictable sometimes. And a value for FF of 160 still seems a little high perhaps. The defaults are 70 for a wing without a tail and only 50 for the general setting. |
I'll try to flash the firmware again and set it up again. After all, others fly fine, but I'm the only one with glitches. |
Maybe I'm not using autotune correctly? |
@kasatka60 You have the logging rate at 50%.. You won't get the F405SE to log reliably above 13%. Especially with If you want a comparison. These are the tunes on one of my Dolphins... With my other Dolphins tune, not being much different.
|
These are the default settings. |
Yeah. Unfortunately SPI card logging experiences issue when logging above 13%.. And even more so with many F405 controllers, especially when more demanding tasks are placed on the controller. |
I'll try to fly tomorrow. Maybe the black box will record logs normally. |
for F722 what black box settings should be set? |
3% (30Hz) would already be enough for this. But do not go above 50% (500Hz) |
Have a nice day everyone |
https://youtu.be/4Bdrkk84Qig |
Autotune sets high ff values and that's why there are problems |
Depending on physical setup. FF below 200 is fine. It could also be how the autotune was performed. |
Next weekend I'll try to fly with these parameters: fw_ff_roll = 37 |
If I understand correctly, because of the large value ff, the plane flies incorrectly |
I'll try to install inav 8 on z-84 and reptile 1100 (There is also T1 vtol) |
If the plane was autotuned correctly. The rates and FF should be correct for your plane's setup. It would be based on the physical throws of your individual aircraft. |
Why does the 8 inava fly so crookedly? inav 7 flew perfectly. |
Just a quick reply.. I looked at your video. But haven't had the time to look at the log. From the way you did the autotune. It was either the control surfaces have very little throws. Or you weren't moving the stick to full for long enough. So it wasn't providing autotune a good snap shot.
The velocity z controller that INAV 8 uses works differently to what INAV 7 used. Meaning the
I spoke to you about how the two are tied together in the lower part of this post. #10536 (comment) |
I will try to follow your advice. I remembered. There is a tab Adjustments. I will try to select the values of nav_fw_pos_z_p during the flight
For autotun, is it correct to make full turns on both axes? |
Hey, I believe we've met on FB Inav page. Altus is at the moment back on 7.1.2 hoping for a short flight tomorrow to see if it's solid on that FW. |
As everyone here recommends: you need to reduce the value of nav_fw_pos_z_p. |
I flew RC2 on both "stock" and recommended by Marc changes on some values. They didn't change anything. |
https://youtu.be/4Bdrkk84Qig |
@kasatka60 @jaroszmm There is also another setting related to the Z velocity controller called Decreasing its value can help prevent altitude target overshoot or oscillations around the target, which may occur on some airframes, if you happen to have If these don't stop it. Which is slightly below what I used at 3C in my winter. Then its another issue that is also adding to this problem. |
@Jetrell Video: time refers to flight timer in te OSD Link to the DVR footage: https://youtu.be/FUuB_qc4uIg?si=MtJhDsLLcegm-Ehv |
LOG00001.TXT Before that I was using default values that either came with 8 or were already in 7 (don't know if they are in DIFF but I guess not) and it was ALSO pitching up and down. I want now to fly again on 7 just to make absolutely sure it is fine on 7. |
The main problem here seems to be related to the position estimations which, looking at the @helicocrasher log in particular, are inconsistent with the sensor data for the Baro, GPS and Accelerometer. There's pretty poor correlation between the Baro and the GPS altitudes and the estimated position and vertical velocity. e.g. between 1:05 and 1:25 in the above log file: Then there's a major glitch at 1:45 when the Position suddenly drops by 26+ meters seemingly because the vertical velocity plummets even though this is contradicted by both the Baro and GPS altitudes which actually increase. Although the vertical acceleration at this point decreases which would correspond with a drop. 3s later the opposite happens and position shoots back up. This may have been caused by the GPS vertical velocity decreasing even though the altitude seemed to keep increasing. Can't tell from the video if this occurred during a stall, didn't look like it and attitude says not but the speed was very low. I guess the problem is which sensors do you believe. One thing to try is to change |
The idea of Baro having issues and messing actually starts to make sense. While on Swordfish that flew okay with 8.0 RC2 I get altitude on OSD instantly and it shows pretty accurate values. The boards are Speedybee 405Mini Wing (the bumpy plane) and Matek 405WMN (the solid one) |
Copied the settings (diff all) from inav 7.1.2 and pasted them into inav 8.0 RC3.
In navigation modes the plane flies unstably.
https://youtu.be/n69w9CDaM8Q
(OSD is a little out of sync)
There is no SD card on board.
Maybe I configured something incorrectly when transferring the settings?
INAV_7.1.2_cli_DOLPHIN_20241215_124916.txt
INAV_8.0.0_cli_DOLPHIN_20241215_150820.txt
The text was updated successfully, but these errors were encountered: