Skip to content

Wrong logic excluding event codes with just one event in read_raw_egi #11626

@jackz314

Description

@jackz314

Description of the problem

I believe when creating a RawMFF object, the logic for excluding events with just one event is wrong (line 434), it rejects them if event.sum() <= 1, but event.sum() could be more than 1 since it contains the event id (1 to n_events), a more appropriate to do this would be something like np.count_nonzero(event) <= 1

Also, in the documentation for read_raw_egi, which uses RawMFF, it says

The event channels to be ignored when creating the synthetic trigger. Defaults to None. If None, channels that have more than one event and the sync and TREV channels will be ignored.

which I believe is also wrong.

I can submit a PR once this is confirmed.

Related, I think it might be helpful to keep events that only occur once, they could be important tags that indicate start/end of experiments for example, a suitable place might be in the annotations.

Steps to reproduce

Import raw mff files

Link to data

No response

Expected results

All one-shot events excluded

Actual results

Only the first one-shot event (with ID 0) is excluded

Additional information

Draft PR #11627

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions