Skip to content

fix TdtRawIO multi-block SEV file lookup (wrong path + case mismatch)#1815

Open
ree-gupta wants to merge 1 commit intoNeuralEnsemble:masterfrom
ree-gupta:fix/tdt-multiblock-sev-lookup
Open

fix TdtRawIO multi-block SEV file lookup (wrong path + case mismatch)#1815
ree-gupta wants to merge 1 commit intoNeuralEnsemble:masterfrom
ree-gupta:fix/tdt-multiblock-sev-lookup

Conversation

@ree-gupta
Copy link

Fixes multi-block SEV file lookup in TdtRawIO. Two bugs caused silent fallback to TEV, resulting in all channels reading identical data:

  1. path variable at L290 was stale (from iterdir() loop at L103 or reassigned to tank root at L170), never pointed to the block subdirectory where SEV files live
  2. Filename used lowercase ch but TDT writes Ch, not found on case-sensitive filesystems (Linux)

Fix: use self.dirname / segment_name for the correct block directory, and [cC]h glob to match both cases (same pattern the single-block path already uses).

Not caught by tests because aep_05 (the only multi-block test dataset) has no SEV files.

Closes #1814

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.

TdtRawIO: multi-block SEV files not found (wrong path + case mismatch)

1 participant