add geo_bound aggregation #1271

Merged
merged 2 commits into from Mar 15, 2017

Conversation

Projects
None yet
4 participants
@cobak78
Contributor

cobak78 commented Mar 15, 2017

add geo_bound aggregation feature issued on #1270

@ruflin

This comment has been minimized.

Show comment
Hide comment
@ruflin

ruflin Mar 15, 2017

Owner

@cobak78 Could you update the changelog?

Owner

ruflin commented Mar 15, 2017

@cobak78 Could you update the changelog?

@ruflin

ruflin approved these changes Mar 15, 2017

@ruflin ruflin merged commit c51b030 into ruflin:master Mar 15, 2017

3 checks passed

codecov/patch 100% of diff hit (target 84.32%)
Details
codecov/project 84.35% (+0.02%) compared to 44782d0
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@ruflin

This comment has been minimized.

Show comment
Hide comment
@ruflin

ruflin Mar 15, 2017

Owner

@cobak78 Merged. Thanks for the contribution.

Owner

ruflin commented Mar 15, 2017

@cobak78 Merged. Thanks for the contribution.

@Tobion

This comment has been minimized.

Show comment
Hide comment
@Tobion

Tobion Mar 15, 2017

Collaborator

The test file is now in the wrong place.

Collaborator

Tobion commented Mar 15, 2017

The test file is now in the wrong place.

@ruflin

This comment has been minimized.

Show comment
Hide comment
@ruflin

ruflin Mar 15, 2017

Owner

:-( Missed that one. @cobak78 Could you open a follow up pr?

Owner

ruflin commented Mar 15, 2017

:-( Missed that one. @cobak78 Could you open a follow up pr?

@Tobion

This comment has been minimized.

Show comment
Hide comment
@Tobion

Tobion Mar 15, 2017

Collaborator
Collaborator

Tobion commented Mar 15, 2017

@cobak78

This comment has been minimized.

Show comment
Hide comment
@cobak78

cobak78 Mar 16, 2017

Contributor

so, it's fixed on #1272?

Contributor

cobak78 commented Mar 16, 2017

so, it's fixed on #1272?

@ruflin

This comment has been minimized.

Show comment
Hide comment
@ruflin

ruflin Mar 17, 2017

Owner

@cobak78 Correct.

Owner

ruflin commented Mar 17, 2017

@cobak78 Correct.

mhernik pushed a commit to mhernik/Elastica that referenced this pull request Jul 24, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment