search

lina-usc / pylossless

🧠 EEG Processing pipeline that annotates continuous data
https://pylossless.readthedocs.io/en/latest/
MIT License
18 stars 8 forks source link

Bug in `flag_ch_bridge` ? #81

Closed scott-huberty closed 1 year ago

scott-huberty commented 1 year ago

If I make a simulated eeg file (with a healthy amount of random noise) - and make channels 19 and 20 duplicates of each other. And confirm that they are perfectly correlated. flag_ch_bridge will not flag these channels - but WILL flag some other channels.

image.

I think this warrants diagram and code review to confirm that the code is behaving as our diagrams say it should.

scott-huberty commented 1 year ago

Note to self: maybe I am making a false assumption that these channels (19 and 20) are adjacent.

Check montage.

scott-huberty commented 1 year ago

yep... Looks like 19 and 20 aren't "nearest neighbors" after all - my mistake :sweat_smile:

Sensor_positions_(eeg)

I'll close this soon.