-
-
Notifications
You must be signed in to change notification settings - Fork 433
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
Allow custom filter to be passed in to constructors. Add SmoothDampFilter as an alternative. #338
base: master
Are you sure you want to change the base?
Allow custom filter to be passed in to constructors. Add SmoothDampFilter as an alternative. #338
Conversation
… filter. (Defaults to OneEuroFilter) Added SmoothDamp implementation.
Hi @jmschrack, thanks for the hard work and sorry for the delay. I finally got a chance to try it out. I realized there is a pretty big delay in the smoothDamp filter (using your default 0.1 smoothTime). If I tune the value down, it becomes jittery as well, but that make sense. I'm trying to replicate a similar "jitter-delay" tradeoff with OneEuro. With beta=0.001, and minCutOff=0.001, it looks like the result is similar. Demo: https://youtube.com/shorts/6uEftY6tr78 |
@hiukim Re: "pretty big delay in smoothDamp" I also tried benchmarking the two filters, and it looks like OneEuroFilter and SmoothDampFilter are roughly equivalent in terms of speed when it comes to operating on 16 element Matrices. (I used c++ to compare the two, since JS is notoriously tricky to get accurate readings.) Normally the two filters are pretty close to each other in timing with the occasional swings one way or the other. |
@hiukim Any more thoughts on this? |
Test is for personal project, smooth looks more pretty. Thanks for suggestion! |
The result in the video looks pretty amazing to me and an obvious improvement. I really wish this was a potential option in Mind AR or at least had an option to provide your own custom filters. In my case, the default filter does nearly nothing, my image tracking is quite annoying to watch because of the amount of jitter I am getting, MinCF and Beta settings have no impact. @hiukim any chance this could be considered again? @jmschrack any potential instructions on how one could utilize your code in case this won't be merged? Really excellent work on this implementation! |
…hancement/customFilter_smoothDamp
@Ucodia before going to a custom solution, have you tried If that doesn't work and you want to try my custom filter setup, uninstall mindar from your project, and then install Caveat: I just pulled in the master and fixed merge conflicts, but I don't have time right now to test it. The main difference between the two filters is that OneEuroFilter can smooth out problems 90% of the time and is extremely performant. But it's that 10% that can seem awful. Whereas SmoothDamp is guaranteed smooth at the cost of increased overhead. However, when i've benchmarked the two against each other, SmoothDamping comes in pretty close to OneEuroFilter. (But benchmarking javascript is never super accurate.) |
@jmschrack I had attempted to use custom values for those parameters but I guess I was not using the appropriate combination. Setting |
three-filter.html
to show passing custom filters.