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

Update coord2area_def.py #931

Merged
merged 4 commits into from Dec 28, 2019
Merged

Update coord2area_def.py #931

merged 4 commits into from Dec 28, 2019

Conversation

AppLEaDaY
Copy link
Contributor

coord2area_def.py update

  • Adjusted commented text to point to the output actually supplied nowadays (yaml format and not the old format).

  • Added instruction in commented text concerning how to display the defined area relying on gshhs library.

  • Removed unused commented lines.

@coveralls
Copy link

coveralls commented Oct 9, 2019

Coverage Status

Coverage increased (+1.9%) to 87.336% when pulling 4978580 on AppLEaDaY:master into a6ce4df on pytroll:master.

@codecov
Copy link

codecov bot commented Oct 9, 2019

Codecov Report

Merging #931 into master will increase coverage by 1.92%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #931      +/-   ##
==========================================
+ Coverage   85.41%   87.33%   +1.92%     
==========================================
  Files         172      188      +16     
  Lines       26046    28624    +2578     
==========================================
+ Hits        22246    24999    +2753     
+ Misses       3800     3625     -175
Impacted Files Coverage Δ
satpy/tests/utils.py 93.87% <0%> (-3.21%) ⬇️
satpy/resample.py 90.61% <0%> (-2.37%) ⬇️
satpy/config.py 80.86% <0%> (-1.12%) ⬇️
satpy/readers/seviri_l1b_nc.py 14.54% <0%> (-0.84%) ⬇️
satpy/writers/cf_writer.py 90.67% <0%> (-0.77%) ⬇️
satpy/scene.py 89.8% <0%> (-0.73%) ⬇️
satpy/readers/nucaps.py 93.71% <0%> (-0.53%) ⬇️
satpy/readers/hdf5_utils.py 91.93% <0%> (-0.38%) ⬇️
satpy/readers/ahi_hsd.py 96.58% <0%> (-0.35%) ⬇️
satpy/tests/reader_tests/test_agri_l1.py 99.01% <0%> (-0.33%) ⬇️
... and 86 more

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 a6ce4df...4978580. Read the comment docs.

Copy link
Member

@mraspaud mraspaud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot for this greatly needed update !
I just have one requested change (on top of the sticker comments), but otherwise it looks good to me !

os.environ['PPP_CONFIG_DIR'] = '/my_local_path/for_satpy_configuration'

As a further functionality this script may give a quick display of the defined area,
provided the path for the GSHHG library is supplied via the "-s" option.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It should maybe be mentioned that pycoast, pillow and aggdraw need to be installed also for this functionality to work.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just done. Thanks for the suggestion!

Copy link
Contributor Author

@AppLEaDaY AppLEaDaY left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Specified module requirements as suggested.

os.environ['PPP_CONFIG_DIR'] = '/my_local_path/for_satpy_configuration'

As a further functionality this script may give a quick display of the defined area,
provided the path for the GSHHG library is supplied via the "-s" option.
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just done. Thanks for the suggestion!

@mraspaud mraspaud merged commit 3339870 into pytroll:master Dec 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants