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 crude stretch not calculating min/max per-band #33

Merged
merged 2 commits into from
Dec 18, 2018

Conversation

djhoese
Copy link
Member

@djhoese djhoese commented Dec 17, 2018

Doing data_arr.min() computes the min/max across all bands together. We want this separated out so we end up with a minimum value for each band separately.

  • Tests added (for all bug fixes or enhancements)
  • Tests passed (for all non-documentation changes)
  • Passes git diff origin/master **/*py | flake8 --diff (remove if you did not edit any Python files)
  • Fully documented (remove if this change should not be visible to users, e.g., if it is an internal clean-up, or if this is part of a larger project that will be documented later)

@djhoese djhoese added the bug label Dec 17, 2018
@djhoese djhoese self-assigned this Dec 17, 2018
@coveralls
Copy link

Coverage Status

Coverage increased (+0.04%) to 86.925% when pulling 359a730 on djhoese:bugfix-crude-minmax into c89ba18 on pytroll:master.

@djhoese djhoese merged commit 982ef88 into pytroll:master Dec 18, 2018
@djhoese djhoese deleted the bugfix-crude-minmax branch December 18, 2018 13:03
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.

2 participants