Expected values from find_edf_events: missing annotations for EDF+D, but not EDF+C in 0.18
See original GitHub issueUsing MacOSX and conda environment, I downloaded two versions of MNE. Using mne==0.17.2 and 0.18 current version, you take two sets of EDF files (EDF+D and EDF+C). They can be manually converted using edfbrowser.
For the EDF+D file it works for 0.17.2, but not 0.18.
Both versions work when the file is in EDF+C format though. Sample code is below. Test data is available upon request (@larsoner has seen the data though)
import mne
# in these two runs, only 0.17.2 matches for both files
# in 0.18, only the first annotations gets the full annotation list
if __name__ == '__main__':
fpath = "./test_edfc.edf"
raw = mne.io.read_raw_edf(fpath, preload=True)
annotations = mne.events_from_annotations(raw)
print(annotations)
fpath = "./test_edfd.edf"
raw = mne.io.read_raw_edf(fpath, preload=True)
annotations = mne.events_from_annotations(raw)
print(annotations)
Issue Analytics
- State:
- Created 4 years ago
- Comments:24 (24 by maintainers)
Top Results From Across the Web
No results found
Top Related Medium Post
No results found
Top Related StackOverflow Question
No results found
Troubleshoot Live Code
Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free
Top Related Reddit Thread
No results found
Top Related Hackernoon Post
No results found
Top Related Tweet
No results found
Top Related Dev.to Post
No results found
Top Related Hashnode Post
No results found

@larsoner trying to see how I can possibly trim them. Will add them if possible.
@adam2392 is it possible for you to make versions of these files as small as possible (e.g. 1MB) that still show the problem? If so then we can incorporate them into our testing data as a regression test. The current size (100MB+) is too big for us to do this