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

Consider z axis #35

Closed
cholmes opened this issue Nov 21, 2017 · 6 comments
Closed

Consider z axis #35

cholmes opened this issue Nov 21, 2017 · 6 comments
Assignees
Labels
prio: should-have would be very good to have in the release stac-item
Milestone

Comments

@cholmes
Copy link
Contributor

cholmes commented Nov 21, 2017

We should consider if height is also part of geometries in STAC, and either explicitly say it's not or make sure that an extension mechanism can handle it, or that it's just ok with geojson definitions.

@m-mohr
Copy link
Collaborator

m-mohr commented Aug 24, 2018

I think that could be an important one if we want to cater for all domains especially such as meteorology.

For the datasets we specified a dimensions extension. I am not sure whether that really fulfills all needs yet. What about items? It seems like the new API spec supports the z-axis already, at least I found references for that during my review of #167.

I already heard voices within openEO that were not very comfortable with GeoJSON as it doesn't allow the z-axis to be specified.

Do we have any expert that we could consult regarding this?

@matthewhanson
Copy link
Collaborator

GeoJSON does allow for an optional 3rd dimension to allow for altitude @m-mohr and this will be needed for a point cloud extension cc @hobu

@m-mohr
Copy link
Collaborator

m-mohr commented Aug 24, 2018

Oh, interesting, thanks for pointing that out. Never saw that in the wild yet, but you are certainly right I just looked it up directly in the spec/RFC.

@m-mohr
Copy link
Collaborator

m-mohr commented Sep 5, 2018

The z-axis has been added to the dataset spec with #239. We should consider adding similar things to the item.

@m-mohr m-mohr added this to the future milestone Jul 22, 2019
@matthewhanson
Copy link
Collaborator

There is no reason to not allow a 3rd dimension in the Item, and I think we do because it's a GeoJSON geometry which allows for it.
However, our description of Geometry and Bounding Box should be updated to say not just lat/lon, but lat/lon/

I'm moving this to 0.8.0 as it's just a clarification.

@matthewhanson matthewhanson modified the milestones: future, 0.8.0 Aug 15, 2019
@matthewhanson matthewhanson added the prio: should-have would be very good to have in the release label Aug 15, 2019
@m-mohr
Copy link
Collaborator

m-mohr commented Aug 16, 2019

Agreed, we should also check the descriptions for spatial collection extents.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
prio: should-have would be very good to have in the release stac-item
Projects
None yet
Development

No branches or pull requests

4 participants