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

Avro Schema Creation for ProjectedExtent and TemporalProjectedExtent #1971

Merged
merged 9 commits into from
Jan 23, 2017

Conversation

jbouffard
Copy link
Contributor

This PR adds two new codecs that can be encode/decoded in GeoTrellis: ProjectedExtent and TemporalProjectedExtent. The main reason for adding these two codecs is to allow for backend support between GeoTrellis and GeoPySpark.

Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
Signed-off-by: jbouffard <jbouffard@azavea.com>
@jbouffard jbouffard changed the title Avro Schema Creation for ProjectedExtent and TemporalProjectedExtent Avro Schema Creation for ProjectedExtent and TemporalProjectedExtent Jan 18, 2017
@jbouffard
Copy link
Contributor Author

@lossyrob This is ready for review.

.record("ProjectedExtent").namespace("geotrellis.vector")
.fields()
.name("extent").`type`(extentCodec.schema).noDefault()
.name("epsg").`type`().intType().noDefault()
Copy link
Member

Choose a reason for hiding this comment

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

Can you update to use the proj4 string.
Or, if possible, check if it's an EPSG code, and then fall back to CRS?

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