Skip to content

MRG: width/height in offscreen mode - #265

Merged
agramfort merged 8 commits into
nipy:masterfrom
larsoner:wh
Mar 22, 2019
Merged

MRG: width/height in offscreen mode#265
agramfort merged 8 commits into
nipy:masterfrom
larsoner:wh

Conversation

@larsoner

Copy link
Copy Markdown
Contributor

Fixes a bug from #245 that showed up when w != h as values ended up swapped.

@codecov-io

codecov-io commented Mar 22, 2019

Copy link
Copy Markdown

Codecov Report

Merging #265 into master will increase coverage by 0.05%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master   #265      +/-   ##
========================================
+ Coverage   74.95%    75%   +0.05%     
========================================
  Files           7      7              
  Lines        2511   2517       +6     
  Branches      505    506       +1     
========================================
+ Hits         1882   1888       +6     
  Misses        454    454              
  Partials      175    175

@larsoner larsoner changed the title FIX: width/height in offscreen mode MRG: width/height in offscreen mode Mar 22, 2019
@larsoner

Copy link
Copy Markdown
Contributor Author

Ready for review/merge from my end. This one is a pretty trivial fix aside from the necessary CI updating. I'll merge tomorrow probably assuming it comes back green unless someone wants more time to look.

@agramfort
agramfort merged commit df68527 into nipy:master Mar 22, 2019
@agramfort

Copy link
Copy Markdown
Contributor

thx @larsoner

@larsoner
larsoner deleted the wh branch March 22, 2019 09:52
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.

3 participants