Skip to content

Conversation

@jona-sassenhagen
Copy link
Contributor

Like #6186, but for time series.

@codecov
Copy link

codecov bot commented Apr 23, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@9cc5d14). Click here to learn what that means.
The diff coverage is 81.65%.

@@            Coverage Diff            @@
##             master    #6189   +/-   ##
=========================================
  Coverage          ?   88.98%           
=========================================
  Files             ?      413           
  Lines             ?    74213           
  Branches          ?    12293           
=========================================
  Hits              ?    66035           
  Misses            ?     5280           
  Partials          ?     2898

@jona-sassenhagen
Copy link
Contributor Author

Should actually be ready now. @drammock can you look at this regrettably boring PR?

@larsoner
Copy link
Member

I am confused, your words say one thing but your PR title says another

@jona-sassenhagen jona-sassenhagen changed the title [WIP, BORING] Refactor ts plotter [NOW IT'S ONLY BORING] Refactor ts plotter Apr 23, 2019
@larsoner larsoner changed the title [NOW IT'S ONLY BORING] Refactor ts plotter MRG: [NOW IT'S ONLY BORING] Refactor ts plotter Apr 23, 2019
@jona-sassenhagen jona-sassenhagen changed the title MRG: [NOW IT'S ONLY BORING] Refactor ts plotter MRG: Refactor ts plotter Apr 23, 2019
@agramfort agramfort merged commit ffa71db into mne-tools:master Apr 24, 2019
@agramfort
Copy link
Member

thx @jona-sassenhagen

jeythekey pushed a commit to jeythekey/mne-python that referenced this pull request Apr 27, 2019
* ...

* ...

* ...

* nm [skip ci]

* update

* ...

* ...

* pep8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants