You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@yardencsGitHub points out 'Hz' is confusing since it is a frequency -- I used it because I wanted something quick to type (and I'm a bad physicist who abuses units of measure)
Maybe 'onset_sample_ind' and 'offset_sample_ind'? 'onset_index' and 'offset_index'?
This is one of those cases where there's no good short name.
'onset_ind' and 'offset_ind' might have to suffice and then for clarity it would need to be clearly defined in docstrings, examples, etc.
The text was updated successfully, but these errors were encountered:
@yardencsGitHub points out 'Hz' is confusing since it is a frequency -- I used it because I wanted something quick to type (and I'm a bad physicist who abuses units of measure)
Maybe 'onset_sample_ind' and 'offset_sample_ind'? 'onset_index' and 'offset_index'?
This is one of those cases where there's no good short name.
'onset_ind' and 'offset_ind' might have to suffice and then for clarity it would need to be clearly defined in docstrings, examples, etc.
The text was updated successfully, but these errors were encountered: