search

LorenFrankLab / rec_to_nwb

Data Migration REC -> NWB 2.0 Service
Other
2 stars 8 forks source link

get_electrode_indices() is incorrect #14

Closed lfrank closed 3 years ago

lfrank commented 3 years ago

This function does not return the indices of electrode by their IDs in its current form and it needs to be fixed. The fix either involves sending in the nwbfile itself so that the IDs can be looked up in the main NWB electrode table or somehow getting those IDs from the electrode table region in the electrical series, but I don't know how to do that.

This will cause LFP extraction to be incorrect in some cases where there are bad channels, so it needs to be fixed asap.

lfrank commented 3 years ago

This is now fixed (thanks to Ryan)