Skip to content

Backport np helper functions to geotrellis.util.np - #3067

Merged
pomadchin merged 1 commit into
locationtech:masterfrom
CloudNiner:feature/awf/backport-math-functions
Sep 6, 2019
Merged

pomadchin merged 1 commit into
locationtech:masterfrom
CloudNiner:feature/awf/backport-math-functions

Conversation

@CloudNiner

@CloudNiner CloudNiner commented Sep 5, 2019 •

Copy link
Copy Markdown
Contributor

Also adds implicit helper methods for Tile.percentile

  • docs/CHANGELOG.rst updated, if necessary
  • docs guides update, if necessary
  • New user API has useful Scaladoc strings
  • Unit tests added for bug-fix or new feature

Closes azavea/geotrellis#122

@CloudNiner
CloudNiner force-pushed the feature/awf/backport-math-functions branch from fee82a4 to b7ce989 Compare September 5, 2019 20:00
@CloudNiner CloudNiner changed the title [WIP] Backport np helper functions to geotrellis.util.np Backport np helper functions to geotrellis.util.np Sep 5, 2019
@CloudNiner
CloudNiner requested a review from pomadchin September 5, 2019 20:01
Comment thread util/src/main/scala/geotrellis/util/np/package.scala Outdated
@CloudNiner
CloudNiner force-pushed the feature/awf/backport-math-functions branch from 527490b to d67a1b7 Compare September 6, 2019 14:07

@pomadchin pomadchin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM! Thanks! Will merge once travis is happy.

@CloudNiner

Copy link
Copy Markdown
Contributor Author

👍

@pomadchin
pomadchin merged commit 7f532fd into locationtech:master Sep 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants