Skip to content

MRG: width/height in offscreen mode #265

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 8 commits into from
Mar 22, 2019
Merged

MRG: width/height in offscreen mode #265

merged 8 commits into from
Mar 22, 2019

Conversation

larsoner
Copy link
Contributor

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

@codecov-io
Copy link

codecov-io commented Mar 22, 2019

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
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
Contributor

thx @larsoner

@larsoner larsoner deleted the wh branch March 22, 2019 09:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants