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
Combine Accela Filter with Hamilton Filter #1843
Conversation
Great job! |
Try explicitly clamping the position value to be between Should I come up with a |
9f75c29
to
8c34de3
Compare
What? Where?
Nah. we can get way with float, I think. The mixing of float and double was awkward though. So I changed everything to float. |
Wasn't |
Hmm .... its the distance where the smoothing assumes the maximum value. We can take our time here until you could test the filter yourself. |
I see I'm added as a reviewer on this. I'm super rusty on this code and probably not the right person to review this. I had a glance... It looks OK to me, but I didn't look very closely. |
@dbaarda Thanks for taking a look even if superficial. sthalik and GO63-samara are available for reviews after all. So let's just merge it. |
As promised a while ago, here is a filter which combines the Accela gain curves with the rotation-space based filtering from the Hamilton filter. I hope it could maybe replace both the Accela and Hamilton filters.
Basically it applies the accela gain to the rotation vector magnitude of the shortest rotation between the current orientation and the measured orientation.
The position calculation is like in the Accela Filter.
I also included a smoothing amplification depending on the z-position. In my version I add to the smoothing amount. In the Hamilton Filter, I think the gain curve was modified which I wouldn't know how to do in the Accela way.
Maybe I could delete some of the disabled code for a spline GUI. Then again, it might be useful for tweaking those gain curves to those who know what they are doing :-D