-
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
Proposed fix for "dolphining" bug in 6.1 #9111
Conversation
…eIMUTopic() to account for HITL and SITL arming flags.
@ricardodeazambuja and @M0j0Risin if you’re both happy with this, and feel it is tested and working as expected. I’ll get this merged and let Paweł know that 6.1.1 is ready. Thanks for your work 👍🏻 |
sadly, I won't be able to test it next week (or the week after), so I will leave to @M0j0Risin. |
@MrD-RC I've been flying it like this for the past two weekends and haven't seen any hint of the issue during nav modes. So I'm fairly confident this corrects the problem. It's a pretty safe bet since all we're really doing is bringing 6.1.1 back on par with 6.0 in this regard. |
I pulled down the 6.1.1 branch and flight tested it this evening. Looking good. |
Never had this issue before on any INAV version but upgraded 2 of my craft recently to 6.1.1 (Dart 250 and Nano Talon) and the Dart 250 is now dolphining like a good one. Nano Talon seems fine so not sure what has changed, diff all from 6.1 was applied to 6.1.1 |
If you have any DVR or blackbox logs. Please open an issue. Replying to merged pull requests is not the right course of action. |
This is completely unrelated. This is not dolphining (slow oscillations on pitch). This is fast roll oscillations. Most likely caused by a bad tune, and too high PIDs. |
Hi Interesting fact, a friends of mine also has the same plane , and was experiencing the same behaviour after cutting the VTail. Here is a blackbox file . I Laowered the Level strenght to 5, ad still have the issue though the oscillation are at lower frequencies And here is a video sampled with the issue: Next time I will disable AUTOLEVELto have another comparison flight, but I doubt this is the source cause because my firend never uses AUTOLEVEL. |
Just as a suggestion to try, set both "Control Smoothness" and "Throttle Smoothing" to 0. These two are under the advance tuning tab-> fixed-wing navigation settings. |
I'll give it atr but throttle smoothing is already '0' and Control Smoothness is '2'. I set those 2 parameters to '0' for next flight to be taken into account. THough they seems to setup throttle only when it is automatically controled and the issue is there whether throttle is manual or not. |
Hello everyone, yesterday I flew in version 6.1.1 with the Matek 405 WMN board, using the main camera and at times the PAN camera. https://www.youtube.com/watch?v=4gHDf2qWOe4 I probably changed to INAV 7 but I wanted to state that the problem has not gone away, it has reappeared in 6.1.1. |
This is essentially just reverting navigation_pos_estimator.c to the version from 6.0. And adding back in the lines to account for the new arming flags related to HITL & SITL added in 6.1.