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

Reintroduce support for pyproj 1.9.6 in cf_writer #1352

Merged
merged 3 commits into from Sep 9, 2020

Conversation

ninahakansson
Copy link
Contributor

@ninahakansson ninahakansson commented Sep 8, 2020

The create_grid_mapping in cf_writer need a newer version of pyproj.
By moving the import of pyproj to that function data with lat/lon can still be written also with pyproj==1.9.6.

  • Tests added
  • Passes flake8 satpy

The create_grid_mapping need a newer version of pyproj,
by moving the import to that function data with lat/lon
can still be written.
@ghost
Copy link

ghost commented Sep 8, 2020

Congratulations 🎉. DeepCode analyzed your code in 2.079 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

@mraspaud
Copy link
Member

mraspaud commented Sep 8, 2020

@ninahakansson I suppose you mean pyproj?

Edit: fixed the title to replace pygrib with pyproj

@mraspaud mraspaud added component:writers enhancement code enhancements, features, improvements labels Sep 8, 2020
@mraspaud mraspaud changed the title Reintroduce support for pygrib 1.9.6 in cf_writer Reintroduce support for pyproj 1.9.6 in cf_writer Sep 8, 2020
@mraspaud
Copy link
Member

mraspaud commented Sep 8, 2020

Retry DeepCode

@mraspaud
Copy link
Member

mraspaud commented Sep 8, 2020

Tests are coming

@codecov
Copy link

codecov bot commented Sep 8, 2020

Codecov Report

Merging #1352 into master will increase coverage by 0.01%.
The diff coverage is 95.23%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1352      +/-   ##
==========================================
+ Coverage   90.26%   90.28%   +0.01%     
==========================================
  Files         226      226              
  Lines       32718    32721       +3     
==========================================
+ Hits        29534    29542       +8     
+ Misses       3184     3179       -5     
Impacted Files Coverage Δ
satpy/writers/cf_writer.py 93.77% <66.66%> (ø)
satpy/tests/writer_tests/test_cf.py 99.59% <100.00%> (+1.01%) ⬆️

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 7f11a0e...c415a94. Read the comment docs.

@coveralls
Copy link

coveralls commented Sep 8, 2020

Coverage Status

Coverage increased (+0.02%) to 90.285% when pulling c415a94 on ninahakansson:cfwriter_support_pyproj_1p9 into 7f11a0e on pytroll:master.

Nina.Hakansson and others added 2 commits September 8, 2020 20:52
@mraspaud mraspaud merged commit 5085b84 into pytroll:master Sep 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:writers enhancement code enhancements, features, improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants