Qian Chu
Qian Chu
> This has been asked before: > > * [Add laplace reference method to mne/io/reference.py #10163](https://github.com/mne-tools/mne-python/issues/10163) > * [Re-reference methods about sEEG data #8481](https://github.com/mne-tools/mne-python/issues/8481) > > If you're willing to...
My colleague @AlexLepauvre is also interested in this topic. Alex, from your perspective (especially having seen different iEEG data from multiple centers) what might be most useful to the community?
To sum the discussion above, it seems that most are in favor of `mne.set_eeg_reference(ref_channels=dict(...))` with certain things to keep in mind: - Make sure to have `inst.info['custom_ref_applied'] = FIFF.FIFFV_MNE_CUSTOM_REF_ON` -...
@AlexLepauvre I've come up with a short to-do list of things we may need to change to complete the PR. Feel free to modify it! My initial commit only changed...
Do you think we need a formal test for the addition (in test_reference)?
Code, tests, and docstring all updated and we are ready for review!
> I've pushed a few commits to tidy up the implementation, and added some inline explanations below in a self-review (the commit messages should hopefully also be helpful in understanding...
@drammock When you have time would you mind re-reviewing? Thanks!
This is a quite old thread but I still want to share my two cents on this. IMO the major reason for restricting keys to be integers is that we...
Maybe `expand()` and `contract()` can take a tuple to include/exclude timepoints before and after. Similar to `numpy.pad`. Also something I think might be useful is to shift the interval as...