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

multifile event handling in csc_convert_from_raw is problematic #3

Open
riedner opened this issue Oct 27, 2014 · 1 comment
Open

multifile event handling in csc_convert_from_raw is problematic #3

riedner opened this issue Oct 27, 2014 · 1 comment

Comments

@riedner
Copy link

riedner commented Oct 27, 2014

Head is created for every file input in a loop (line 22)

Head is used for the eventcodes (line 65) but it is re-created for every file loop (line 22). This will crash if the last file doesn't have the correct eventcodes (e.g. if it doesn't have any events and any other file does)

@Mensen
Copy link
Member

Mensen commented Oct 28, 2014

Yes, I spotted this problem when I tried to use it for my own data recently. In fact this function essentially ignores any events in the data as we used the function for sleep data conversion before.

I have just reverted to the basic EEG = pop_readegi for all the work I need and then EEG = pop_mergeset(...) if I need to concatenate. This way I can import a segment, downsample it and then concatenate which avoids loading a lot of data into RAM at once.

This script is based on eeglab's own pop_readsegegi so if the function is useful we can take more ideas about the event processing from there.

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

2 participants