Skip to content

Added Sum resampling method to raster.resampling - #2326

Merged
echeipesh merged 3 commits into
locationtech:masterfrom
Charmatzis:master
Sep 28, 2017
Merged

echeipesh merged 3 commits into
locationtech:masterfrom
Charmatzis:master

Conversation

@Charmatzis

Copy link
Copy Markdown
Contributor

Does what it says: Sum method has been added for resampling.

Now the aggregate resampling methods are:

  1. Median
  2. Max
  3. Min
  4. Sum

@pomadchin

pomadchin commented Aug 10, 2017 •

Copy link
Copy Markdown
Member

Thanks @Charmatzis! Can you sign your commits?
git commit --amend -s && git push origin -f master

@echeipesh

echeipesh commented Sep 7, 2017 •

Copy link
Copy Markdown
Contributor

@Charmatzis actually just adding Signed-off-by: Christos Charmatzis <[email protected]> to the bottom of the PR is sufficient to make ip-validation happy. I know this is silly, but if you don't mind.

@pomadchin

pomadchin commented Sep 7, 2017 •

Copy link
Copy Markdown
Member

@Charmatzis try to rebase on master, probably IP issue with [email protected] would go:

git commit rebase -i upstream/master && git push origin -f master

@echeipesh there is a valid sign-off-by footer, looks like IP checker result is that email has no valid ECAs. Can you check it manually?

@echeipesh

Copy link
Copy Markdown
Contributor

screen shot 2017-09-28 at 4 36 19 pm

Looks good, blaming ip-validation bot glitch.

@echeipesh
echeipesh merged commit 4c93452 into locationtech:master Sep 28, 2017
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.

3 participants