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

Set track recording buffer to exact length #23

Merged
merged 4 commits into from
Nov 25, 2022

Conversation

ericyd
Copy link
Owner

@ericyd ericyd commented Nov 23, 2022

Using the "recordingLength" from the recording processor does not typically equate to the length of the loop. It is often very close, but not precisely equal.

For one or two loops, the discrepancy is hard to detect, but over many loops the loop drifts substantially behind the beat and it becomes impossible to play along.

This fixes that issue

Comment on lines +140 to +146
// 60 seconds beats 60 seconds minute
// ———————————— ➗ ————— 🟰 ——————————— 𝒙 ——————— =>
// minute minute minute beats
//
// seconds minutes measures beats samples samples
// ————————— 𝒙 ———————— 𝒙 ———————— 𝒙 ———————— 𝒙 ————————— 🟰 ——————————
// minute beat loop measure second loop
Copy link
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

for some reason this doesn't align super well in my editor compared to github... seems like a weird font issue

@ericyd ericyd merged commit f6c39d6 into main Nov 25, 2022
@ericyd ericyd deleted the trim-recording-buffer-to-length branch November 25, 2022 22:11
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.

1 participant