ericcornelissen
commented
7 years ago Closing this since it should be part of #285
Closed jverbraeken closed 7 years ago
ericcornelissen
commented
7 years ago Closing this since it should be part of #285
Relevant Issues: -
Related Pull Requests: -
What
This Pull Request removes RhythmHelper
Why
This Pull Request is needed because RhythmHelper was unused. It was added to verify if the rhythm of the master matched the rhythm of the slave, but it has been replaced by some machine learning that can uses a completely different system based on short / long intervals
How
This feature can be viewed/tested within the project by
Alternative implementation
Other implementations that I've have considered are
Notes