Skip to content

Allow wider range of bpm in drummloop#1

Open
jonnius wants to merge 3 commits intot-mon:masterfrom
jonnius:changes
Open

Allow wider range of bpm in drummloop#1
jonnius wants to merge 3 commits intot-mon:masterfrom
jonnius:changes

Conversation

@jonnius
Copy link
Contributor

@jonnius jonnius commented Jul 28, 2018

  • allow -30 to +30 bpm compared to default bpm
  • make sure, that values can't become negative

@jonnius jonnius force-pushed the changes branch 2 times, most recently from 084b025 to f56a50c Compare July 28, 2018 13:55
@t-mon
Copy link
Owner

t-mon commented Nov 22, 2018

I have tested this and the quality of the drum sounds gets bad for this range. Can you confirm that?

@jonnius
Copy link
Contributor Author

jonnius commented Nov 24, 2018

You are right. On some of the drums the ratio between minimum and measured bpm was to small. I changed the minimum to be relative to the measured bpm. Now it should be fine.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants