Skip to content

Added two commands: set_compression_level and set_vbr_encoding_quality #341

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

gesellkammer
Copy link
Contributor

These allow to configure the compression/quality when writing to a compressed format (ogg or flac)

…y, two configure the compression/quality when writing to a compressed format (ogg or flac)
soundfile.py Outdated
_ffi.new("double*", quality),
_ffi.sizeof("double"))

if err == SF_FALSE
Copy link
Owner

Choose a reason for hiding this comment

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

Missing colon

@bastibe
Copy link
Owner

bastibe commented Jul 7, 2022

It seems that the Windows tests for file descriptors are breaking with this PR. That doesn't seem related, though, as this code is not included in the current tests, nor has any interaction with file descriptors. Do you have any idea what's going on here?

@gesellkammer
Copy link
Contributor Author

I am not familiar with the tests but the errors do not seem to have to do with the changes here. I would be surprised if the unmodified code would not trigger the same errors.

@gesellkammer
Copy link
Contributor Author

Any news regarding the failing tests?

@bastibe
Copy link
Owner

bastibe commented Jul 28, 2022

No idea, sorry.

@eshaz
Copy link

eshaz commented Feb 15, 2025

@bastibe I'm interested in this feature. Any updates on if it can be merged? I also don't see how this code could interact with a Windows file descriptor test.

Edit: looks like this feature was added in #447 recently and released in 0.13.0. However, at this time of writing the https://python-soundfile.readthedocs.io/ is still refereing 0.11.0 as the latest version, so that should probably be updated.

@bastibe
Copy link
Owner

bastibe commented Feb 15, 2025

Thank you for the heads-up. I hadn't noticed that the docs were out of date. I rebuilt the docs, and they should be up-to-date now. The default redirect still points to 0.11.0, apparently, even though I configured it otherwise. Hopefully this will sort itself out on its own.

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.

3 participants