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

Add coordinate (de-)serialisation, including z #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

joprodrigues
Copy link

In addition to all the Geometry types (including point), in some cases the we are interested in (de-)serialising the coordinates themselves.
The deserialisation code aceppts both pairs/triples of coordinates, such as [X,Y, Z], as well as the format {'x': X, 'y': Y, 'z': Z}, where Z is, similarly to JTS, optional and signalled with a NaN.

If the z axis is present in serialisation, i.e. z is not NaN, then
export it in Json. Likewise, if it's present in Json, then create the
Coordinate with the 3 axis.
@autermann
Copy link
Member

autermann commented Feb 11, 2022

Hi @joprodrigues,

thanks a lot for your contribution.

In order to publish your work within the 52N repositories, we need your official permission. For this purpose we have a so called contributors license agreement (CLA) in place. With this agreement you grant us the rights to use and publish your code under an open source license.

The contributors license agreement is available here.

Please note: as you are working on the code as part of your work it is important, that the CLA is signed by an authorized representative of your company.

Further explanations about the contributors license agreement are available here.

Finally, we have also prepared an FAQ about the contributors license agreement.

We are really looking forward to host your code on 52N. If you have any further questions or need for clarification just let us know. We will be happy to help you.

Thanks,
Christian

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