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

Unable to use some WFDB-compliant files #17

Open
matthewreyna opened this issue Sep 4, 2024 · 0 comments
Open

Unable to use some WFDB-compliant files #17

matthewreyna opened this issue Sep 4, 2024 · 0 comments

Comments

@matthewreyna
Copy link
Collaborator

A user reported that they were unable to load WFDB files with .mat file extensions, e.g., from the PhysioNet Challenge 2020:
https://physionet.org/content/challenge-2020/1.0.2/training/georgia/g1/#files-panel

I suspect that the find_records function is causing this problem:

def find_records(folder, output_dir):

There are at least three issues to fix:

  1. This function hard-codes the signal filename extensions. It should load the signal filenames from the header files.
  2. It associates the signal files with header files by sorting the filenames, assuming that the order of the sorted filenames provides a pairing of the header and signal files. It should load the signal filenames from the header files.
  3. It uses platform-specific path separators to separate and join file paths, e.g., / vs \. It should use the built-in functions from os.path.
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

No branches or pull requests

1 participant