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

Support cube:dimensions and overriding of colormap/expression #26

Merged
merged 9 commits into from
Aug 20, 2024

Conversation

jverrydt
Copy link
Contributor

For static datasets, we use the cube:dimensions STAC collection config to store the available time dimensions. So, if this time dimension is available, use this to build the capabilities document instead of start- and enddate of the collection.

Also added ability to override some titiler parameters like expression, colormap and color_formula so if you pass them as query parameters, they will have precedence on the config from STAC.

Small EPSG identifier fix in capabilities.

@jverrydt
Copy link
Contributor Author

@vincentsarago When can we expect this PR to be reviewed / merged? We need this time dimension feature to proceed to an operational environment.

@vincentsarago
Copy link
Member

@jverrydt the overall looks great, it would be nice to have tests written for this before we merge 🙏

@jverrydt
Copy link
Contributor Author

Great, I'll have a look to add some tests by the end of this week.

@jverrydt
Copy link
Contributor Author

There was some delay due to holiday period but I finally managed to add some additional tests :-)

@jverrydt jverrydt closed this Aug 13, 2024
@jverrydt jverrydt reopened this Aug 13, 2024
@jverrydt
Copy link
Contributor Author

When can we plan a release for this?

@vincentsarago
Copy link
Member

working on this today @jverrydt

@vincentsarago vincentsarago merged commit ecddb6d into developmentseed:main Aug 20, 2024
14 checks passed
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.

3 participants