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

Work around "psf_fseek" bug #63

Merged
merged 1 commit into from
May 14, 2024
Merged

Work around "psf_fseek" bug #63

merged 1 commit into from
May 14, 2024

Conversation

bemoody
Copy link
Collaborator

@bemoody bemoody commented May 14, 2024

Something's wrong with the soundfile wheel packages for x86-64 Linux: MIT-LCP/wfdb-python#486

An example is:

waveform_benchmark.py --input_record 82284982 --physionet_directory mimic4wdb/0.1.0/waves/p169/p16955095/82284982 --format_class waveform_benchmark.formats.wfdb.WFDBFormat516

I think this probably not a bug in wfdb. I'm not sure what the problem is exactly.

As a workaround, pin the soundfile package to an older version (https://pypi.org/project/soundfile/0.11.0/).

There seems to be some problem with the soundfile 0.12.0 and 0.12.1
wheel packages for x86-64 Linux - I haven't completely figured out the
cause of the problem.  In some circumstances, when *reading* a FLAC
signal file with wfdb-python, we get the error "internal psf_fseek()
failed".

The easiest way to avoid this is to use an older soundfile version
which doesn't include Linux binary wheels (but does include binary
wheels for Windows and MacOS.)  On Linux, this will use the system
libsndfile binary that you probably already have installed.
@briangow
Copy link
Collaborator

Thanks @bemoody

@briangow briangow merged commit 155b872 into main May 14, 2024
1 check passed
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