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

Document behaviour on default enhancement #2247

Merged
merged 3 commits into from Nov 2, 2022

Conversation

gerritholl
Copy link
Collaborator

Document Satpys behaviour on applying a default enhancement if no enhancement is defined, and document on how to avoid this enhancement from being applied. Also add a recommendation to always define an enhancement even if using the default.

  • Closes #xxxx
  • Tests added
  • Fully documented

Document Satpys behaviour on applying a default enhancement if no
enhancement is defined, and document on how to avoid this enhancement
from being applied.  Also add a recommendation to always define an
enhancement even if using the default.
@gerritholl gerritholl marked this pull request as ready for review October 27, 2022 15:30
@codecov
Copy link

codecov bot commented Nov 1, 2022

Codecov Report

Merging #2247 (8dba3d9) into main (5ff1611) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2247   +/-   ##
=======================================
  Coverage   94.19%   94.19%           
=======================================
  Files         295      295           
  Lines       45376    45376           
=======================================
  Hits        42743    42743           
  Misses       2633     2633           
Flag Coverage Δ
behaviourtests 4.66% <ø> (ø)
unittests 94.84% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@coveralls
Copy link

coveralls commented Nov 1, 2022

Coverage Status

Coverage remained the same at 94.794% when pulling 8dba3d9 on gerritholl:default-stretch into 5ff1611 on pytroll:main.

doc/source/composites.rst Outdated Show resolved Hide resolved
@djhoese djhoese merged commit 05419d7 into pytroll:main Nov 2, 2022
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

3 participants