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

Implement setRadius #55

Merged
merged 1 commit into from
Dec 14, 2019
Merged

Conversation

skinkie
Copy link
Contributor

@skinkie skinkie commented Dec 12, 2019

A projection may have a radius. This commit implements setRadius used in for example Equidistant Cylindrical where:
+R= Radius of the sphere given in meters. As per documentation this function is applied on the end, so it takes precedence over +ellps.

Fix #54

@pomadchin
Copy link
Member

pomadchin commented Dec 12, 2019

Hey @skinkie thanks for your contribution I'll merge it ASAP! And gz with your first contribution into this project!

Could you please sign an ECA? https://github.com/locationtech/proj4j/blob/master/CONTRIBUTING.md#sign-off-on-commits-and-filing-a-cla

After that run in your terminal:

git commit --amend -s && git push origin -f feature_setRadius

A projection may have a radius. This commit implements setRadius used in for example Equidistant Cylindrical where:
+R=<value> Radius of the sphere given in meters. As per documentation this function is applied on the end, so it takes precedence over +ellps.

Signed-off-by: Stefan de Konink <[email protected]>
@skinkie
Copy link
Contributor Author

skinkie commented Dec 12, 2019

@pomadchin it would help me greatly if you could also push out a new release having this on board, so I can directly use it in my own projects.

@skinkie
Copy link
Contributor Author

skinkie commented Dec 14, 2019

@pomadchin is there anything you still require from me?

Copy link
Member

@pomadchin pomadchin left a comment

Choose a reason for hiding this comment

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

LGTM, all tests are passing! Thanks for your contribution!

@pomadchin pomadchin merged commit 96a1fab into locationtech:master Dec 14, 2019
@pomadchin
Copy link
Member

pomadchin commented Dec 17, 2019

hey @skinkie; we plan to publish a new release during the next couple of weeks (I hope before holidays)

Thanks again for contributing!

@skinkie
Copy link
Contributor Author

skinkie commented Dec 17, 2019

Thank you for bringing proj4j back to active development.

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.

Support for parameters like R
2 participants