Implement lulu filtering for QuickSilver#121
Open
pjpei wants to merge 12 commits into
Open
Conversation
This comment was marked as duplicate.
This comment was marked as duplicate.
Sorry, something went wrong.
DusKing1
reviewed
Dec 26, 2024
bkleiner
requested changes
Jan 2, 2025
bkleiner
force-pushed
the
develop
branch
2 times, most recently
from
8378e8e to
8b6c562
Compare
bkleiner
force-pushed
the
lulu
branch
2 times, most recently
from
6bd7e3e to
8ed66b9
Compare
bkleiner
force-pushed
the
develop
branch
5 times, most recently
from
276c753 to
2e71911
Compare
"Midpoint" is expo origin/pivot, not a remap where X=0.5 results in Y=mid
bkleiner
force-pushed
the
develop
branch
2 times, most recently
from
737ada2 to
011b733
Compare
bkleiner
force-pushed
the
develop
branch
2 times, most recently
from
6290bdb to
c3fac20
Compare
bkleiner
force-pushed
the
develop
branch
2 times, most recently
from
f5f57bd to
f101fc8
Compare
bkleiner
force-pushed
the
develop
branch
3 times, most recently
from
f61168d to
72d0555
Compare
bkleiner
force-pushed
the
develop
branch
3 times, most recently
from
77e4b14 to
90645c4
Compare
bkleiner
force-pushed
the
develop
branch
10 times, most recently
from
d64c667 to
15aefae
Compare
bkleiner
force-pushed
the
develop
branch
2 times, most recently
from
11ffe48 to
057ce3f
Compare
bkleiner
force-pushed
the
develop
branch
3 times, most recently
from
67b4829 to
25eb8c7
Compare
bkleiner
force-pushed
the
develop
branch
3 times, most recently
from
0d99960 to
40abae4
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hi! Thanks for considering this contribution!
I have found this filter works well with low latency in combination with a dynamic gyro filter and a single DTerm filter.
The below is a bad spot of the filter shown, but given the latency I believe this is quite nice even with that taken into account. It excels at high throttle but when dropping the throttle from high to low quickly has a slight wobble if the dynamic gyro isn't enabled. If it is enabled, it works great!
The frequency is more of an indication and doesn't represent anything sensible, really. I recommend starting with 160Hz and adjusting by 30hz at a time upwards up to 250 or down to 100 depending on quad size.
This thing is heavy on cpu compared to the PT series filters so be aware of that, but QuickSilver seems to handle that perfectly.
See the inav pull request for some additional info:
iNavFlight/inav#10145
Also let me know if there are any questions or adjustments I need to make still! :)