Skip to content

HEED loader hardcodes HEED path but README uses heed/ #5

Description

@JoeScharpf

The README tells users to put HEED data in data/heed/ and run with --base heed, but src/ecg_datasets/base/heed/heed.py hardcodes DATA_DIR/HEED/... for both glob and open_ecg. On Linux (case-sensitive filesystems) those are different paths, so preprocessing fails even when setup follows the docs.

Fix: use f"{DATA_DIR}/{self.args.base}" consistently (e.g. self.data_root), matching how the CSV path already uses args.base.

Relevant lines in heed.py: glob.glob(f"{DATA_DIR}/HEED/I*") and file_path = f"{DATA_DIR}/HEED/{row['path']}".

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions