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

Add information to Quickstart on basics of getting measurement values and navigation #1018

Merged
merged 5 commits into from Dec 22, 2019

Conversation

rayg-ssec
Copy link
Contributor

Some users just want easy access to the measurement data, and more often than not, lat/lon arrays corresponding to them. This PR adds some basic documentation on calculating measurement values (BT, reflectance, etc), latitude and longitude arrays, and basic bbox and area slicing using the scene.

  • Closes #xxxx
  • Fully documented
  • Add your name to AUTHORS.md if not there already

@coveralls
Copy link

coveralls commented Dec 12, 2019

Coverage Status

Coverage increased (+0.0009%) to 87.371% when pulling 6ed8d2a on rayg-ssec:docs-quickstart-measurement-values into 3ba7b3d on pytroll:master.

@codecov
Copy link

codecov bot commented Dec 12, 2019

Codecov Report

Merging #1018 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1018      +/-   ##
==========================================
+ Coverage   87.37%   87.37%   +<.01%     
==========================================
  Files         183      183              
  Lines       28179    28181       +2     
==========================================
+ Hits        24620    24622       +2     
  Misses       3559     3559
Impacted Files Coverage Δ
satpy/tests/compositor_tests/__init__.py 99.6% <0%> (ø) ⬆️
satpy/composites/__init__.py 78.99% <0%> (+0.04%) ⬆️

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 3ba7b3d...6ed8d2a. 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 for taking the time, this is useful information indeed.

I just think a couple of points should be added here.

>>> vis006_slice_meas = vis006_slice.values
>>> vis006_slice_lon, vis006_slice_lat = vis006_slice.attrs['area'].get_lonlats()

To subset multi-resolution data, use the `.crop` function.
Copy link
Member

@mraspaud mraspaud Dec 16, 2019

Choose a reason for hiding this comment

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

Can you make this a link to the crop method ?

Copy link
Member

Choose a reason for hiding this comment

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

You should be able to do:

:meth:`~satpy.scene.Scene.crop`

The ~ will make it only say "crop" in the actual text.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added.

doc/source/quickstart.rst Show resolved Hide resolved
doc/source/quickstart.rst Show resolved Hide resolved
@mraspaud mraspaud merged commit 73e081f into pytroll:master Dec 22, 2019
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

4 participants