mmagnuski / borsar

Various tools for EEG/MEG data analysis.
https://mmagnuski.github.io/borsar.github.io/index.html
BSD 3-Clause "New" or "Revised" License
9 stars 1 forks source link

Fix freq new mne #113

Closed mmagnuski closed 1 year ago

mmagnuski commented 1 year ago

Hopefully fixes #111.

mmagnuski commented 1 year ago

Thanks for your suggestions @drammock!

mmagnuski commented 1 year ago

Unfortunatelly I pushed some changes before I saw your comments and now, I can't use the "commit suggestion" button so you could be the author of the commit changes.

drammock commented 1 year ago

Unfortunatelly I pushed some changes before I saw your comments and now, I can't use the "commit suggestion" button so you could be the author of the commit changes.

it doesn't really matter to me. If it bothers you though, you can add this to the commit message (on its own line) when you merge:

Co-authored-by: Daniel McCloy <dan@mccloy.info>

see here

codecov[bot] commented 1 year ago

Codecov Report

Merging #113 (c592bb2) into master (55d5e6a) will increase coverage by 0.67%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #113      +/-   ##
==========================================
+ Coverage   97.11%   97.79%   +0.67%     
==========================================
  Files          18       18              
  Lines        2116     2127      +11     
==========================================
+ Hits         2055     2080      +25     
+ Misses         61       47      -14