Skip to content
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

BPM Detector with a button #2

Closed
mercuriete opened this issue Nov 4, 2018 · 2 comments · Fixed by #3
Closed

BPM Detector with a button #2

mercuriete opened this issue Nov 4, 2018 · 2 comments · Fixed by #3
Labels
enhancement New feature or request

Comments

@mercuriete
Copy link
Owner

mercuriete commented Nov 4, 2018

As a < type of user >, I want < some goal > so that < some reason >.
Fuking Scrum

I as a product_owner/analyst/developer/tester/user, I want to do a bpm detector so that i can detect BPM


OK but now serious
part of the job is actually in develop branch

from the point of the user:
The user can tap a button several times and then in a label the BPM should appear in a label

form the point of the implementation:
The logic takes the user input and should save a system clock timestamp in an array.
Then some time later, the app can calculate the average of the deltas between taps and then we can convert from nanoseconds (period) to beat per minute (frequency).

@mercuriete mercuriete mentioned this issue Nov 4, 2018
37 tasks
@mercuriete mercuriete added the enhancement New feature or request label Nov 4, 2018
@mercuriete
Copy link
Owner Author

As detected while developing, we need to do the average with only a few timestamps (per example only 5 taps)
Another improvement can be add to the interface a feature to clean the array of timestamps.

@mercuriete
Copy link
Owner Author

already done in develop branch.
opening a pull request

@mercuriete mercuriete moved this from In progress to Reviewer approved in android-musician-tools kanban Nov 7, 2018
android-musician-tools kanban automation moved this from Reviewer approved to Done Nov 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Development

Successfully merging a pull request may close this issue.

1 participant