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

Big update #1

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Big update #1

wants to merge 10 commits into from

Conversation

myQwil
Copy link

@myQwil myQwil commented Jan 30, 2022

There was a fairly large update recently, involving the implementation of the Secret Rabbit Code resampler, which makes for much smoother speed transitions. Rebuilding for Windows and MacOS still needs to be done before the pull request is ready.

A few other changes were also made:

  • removed pd-lib-builder and added it back as a submodule

  • removed duplicate abstractions

    • duplicates of files can be really cumbersome to maintain. Better to encourage users
      to copy/paste the binary for their OS into the src folder.

This will be a nightmare to maintain. Better to encourage users
to copy/paste the binary for their OS into the src folder.
ffplay now uses Secret Rabbit Code for resampling,
making for much smoother speed transitions.
debug symbols don't really need to be added to release builds
@myQwil myQwil force-pushed the big-update branch 2 times, most recently from 5f67148 to f9950a9 Compare January 30, 2022 21:44
Otherwise, the speed will switch from very slow to very fast
when given a negative value.
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