Skip to content
This repository was archived by the owner on Aug 17, 2021. It is now read-only.

Add PeakSpeed centering algorithm#12

Open
OPNA2608 wants to merge 1 commit intochiptunecafe:masterfrom
OPNA2608:add_peakspeed
Open

Add PeakSpeed centering algorithm#12
OPNA2608 wants to merge 1 commit intochiptunecafe:masterfrom
OPNA2608:add_peakspeed

Conversation

@OPNA2608
Copy link
Member

Prototype implementation of the PeakSpeed centering algorithm. First time doing anything in Rust, please forgive me if i made and mistakes in code style or anything.

Usually, the algorithm likes to have abit of a safety look-behind & look-ahead into the data stream for more stability, which is currently not possible due to a design flaw - as discussed elsewhere. I think it may feel abit wobbly at times because of that, but it more of less works for now.

@rastertail
Copy link
Collaborator

Usually, the algorithm likes to have abit of a safety look-behind & look-ahead into the data stream for more stability, which is currently not possible due to a design flaw

Fixing this is at the very top of my todo list right now, and that will also include redesigning the centering API a little bit, so I'd rather not merge this until I get that done. Porting your current code to my new API idea shouldn't be difficult.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants