Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Fix handling of navigation in a grib file with lons greater than 180 #476

Merged
merged 3 commits into from Oct 26, 2018

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Oct 26, 2018

Grib files are weird. This should hopefully handle more of them a little better.

  • Tests added
  • Tests passed
  • Passes git diff origin/master -- "*py" | flake8 --diff

@djhoese djhoese self-assigned this Oct 26, 2018
@coveralls
Copy link

coveralls commented Oct 26, 2018

Coverage Status

Coverage increased (+0.07%) to 74.102% when pulling dd60146 on djhoese:bugfix-grib-area into be0c391 on pytroll:master.

@codecov
Copy link

codecov bot commented Oct 26, 2018

Codecov Report

Merging #476 into master will increase coverage by 0.07%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master    #476      +/-   ##
=========================================
+ Coverage   74.02%   74.1%   +0.07%     
=========================================
  Files         136     136              
  Lines       18004   18032      +28     
=========================================
+ Hits        13328   13362      +34     
+ Misses       4676    4670       -6
Impacted Files Coverage Δ
satpy/readers/grib.py 81.88% <100%> (+6.29%) ⬆️
satpy/tests/reader_tests/test_grib.py 95.74% <100%> (+0.93%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update be0c391...dd60146. Read the comment docs.

@djhoese djhoese merged commit e2a3ac7 into pytroll:master Oct 26, 2018
@djhoese djhoese deleted the bugfix-grib-area branch October 26, 2018 18:52
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.

None yet

2 participants