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

Time ranges and the "datetime" field #613

Closed
m-mohr opened this issue Oct 23, 2019 · 6 comments
Closed

Time ranges and the "datetime" field #613

m-mohr opened this issue Oct 23, 2019 · 6 comments
Assignees
Milestone

Comments

@m-mohr
Copy link
Collaborator

m-mohr commented Oct 23, 2019

For time ranges we have the datetime-range extension. Unfortunately, it is not clear what to set for the required datetime field in a STAC Item. Should it be the first date-time? Should it be some kind of a middle value?

This issue appears often for the SAR extension/DAR data, videos and timeseries bundled in a single asset.

As I feel the datetime field is somewhat limited or not clearly specified for quite a good amount of data, I'd propose to make the datetime-range extension core and to require either a single datetime in the datetime field or a range with the start_datetime and end_datetime fields. Both could also be used together.

We can easily validate this using JSON schema (anyOf).

What do others think?

@matthewhanson
Copy link
Collaborator

There are some cases where all 3 dates are needed. For instance, you might have a range of datetimes for some data, but there is also a nominal date. MODIS product MCD43A4 is like this where it is a 16 day mosaic but the values have been corrected to represent the middle day in that range.

I think datetime should always be required, the ranges should be optional, and datetime should be the center time of nothing else.

@m-mohr
Copy link
Collaborator Author

m-mohr commented Oct 25, 2019

That's why I wrote:

Both could also be used together.

Why always require a center time though?

@m-mohr
Copy link
Collaborator Author

m-mohr commented Nov 6, 2019

We'll move datetime-range extension to core, remove the prefix.
datetime will still be required, datetime-range optional.
Extensions (e.g SAR, Video) that use datetime-range must say what datetime is (central, start, end, ...) (This is the todo here.)

@m-mohr
Copy link
Collaborator Author

m-mohr commented Nov 13, 2019

It still feels that by requiring a single instance in time we are moving a server implementation issue to the spec. I've now seen this another time in the PR for Tiled Assets #614 that the datetime just doesn't seem properly used where a datetime range would be more applicable. And there are so many more use cases (timeseries, videos, mosaics, ...).

@m-mohr
Copy link
Collaborator Author

m-mohr commented Apr 29, 2020

Recent discussions lead to the idea to allow null for datetime if start_datetime and end_datetime are specified.

@m-mohr m-mohr modified the milestones: 0.9.0-RC1, 1.0.0-beta1 Apr 29, 2020
@m-mohr m-mohr self-assigned this Apr 29, 2020
@m-mohr
Copy link
Collaborator Author

m-mohr commented May 4, 2020

Will probably be tackled by #792

@m-mohr m-mohr closed this as completed May 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants