Skip to content

MRG: Fix inconsistencies with compute_proj_raw#6160

Merged
larsoner merged 2 commits intomne-tools:masterfrom
larsoner:fix-proj-raw
Apr 17, 2019
Merged

MRG: Fix inconsistencies with compute_proj_raw#6160
larsoner merged 2 commits intomne-tools:masterfrom
larsoner:fix-proj-raw

Conversation

@larsoner
Copy link
Copy Markdown
Member

Makes duration=None and duration=1. (default) equivalent when possible. Bugs:

  1. baseline=None should have been used in Epochs constructor
  2. proj=False should have been used in Epochs constructor
  3. tmax=duration - 1. / sfreq should have been used in Epochs constructor
  4. More careful handling of duration for non-integer sample rates

@codecov
Copy link
Copy Markdown

codecov bot commented Apr 17, 2019

Codecov Report

Merging #6160 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master   #6160      +/-   ##
=========================================
+ Coverage      89%     89%   +<.01%     
=========================================
  Files         410     410              
  Lines       73834   73858      +24     
  Branches    12250   12253       +3     
=========================================
+ Hits        65713   65737      +24     
  Misses       5226    5226              
  Partials     2895    2895

@larsoner larsoner merged commit b3183fc into mne-tools:master Apr 17, 2019
@larsoner larsoner deleted the fix-proj-raw branch April 17, 2019 18:40
massich pushed a commit to massich/mne-python that referenced this pull request Apr 23, 2019
* FIX: Fix inconsistencies with compute_proj_raw

* BUG: SciPy req
jeythekey pushed a commit to jeythekey/mne-python that referenced this pull request Apr 27, 2019
* FIX: Fix inconsistencies with compute_proj_raw

* BUG: SciPy req
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants