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

[turf] Add turf area measurement method. #1079

Merged
merged 5 commits into from
Sep 24, 2019
Merged

Conversation

pengdev
Copy link
Member

@pengdev pengdev commented Sep 19, 2019

Resolves #1067 by porting turf-area from TurfJs to java.

@pengdev
Copy link
Member Author

pengdev commented Sep 23, 2019

@langsmith I have marked the pr as since 4.9.0, maybe need to update to 4.10.0 ?

@langsmith
Copy link

Yes, please @pengdev .

@pengdev
Copy link
Member Author

pengdev commented Sep 24, 2019

Yes, please @pengdev .

Thanks, now it's changed to 4.10.0

Copy link
Member

@tobrun tobrun left a comment

Choose a reason for hiding this comment

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

This looks good @pengdev! Thank you for porting the test suite and keeping the implementation close to the JS one!

@pengdev pengdev merged commit 9739241 into master Sep 24, 2019
@pengdev pengdev deleted the peng-turf-add-area-method branch September 24, 2019 09:32
@langsmith langsmith mentioned this pull request Jan 8, 2020
15 tasks
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.

Add Turf Area method
3 participants