-
Notifications
You must be signed in to change notification settings - Fork 6
Update for titiler 1.0 #41
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
base: main
Are you sure you want to change the base?
Conversation
|
|
||
|
|
||
| @attr.s | ||
| class CustomSTACReader(MultiBaseReader): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
moved and renamed to titiler.stacapi.reader.SimpleSTACReader
| """Post Init.""" | ||
| self.input = self.url | ||
| if bbox := self.input.get("bbox"): | ||
| self.bounds = tuple(bbox) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
having search in the backend construction enable bounds definition for the mosaic
| retry_config = RetrySettings() | ||
|
|
||
|
|
||
| def accept_media_type(accept: str, mediatypes: List[MediaType]) -> Optional[MediaType]: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
now in titiler.core
| datetime=datetime, | ||
| filter_expr=filter_expr, | ||
| filter_lang=filter_lang, | ||
| ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
a lot of code duplication between SearchParams and CollectionSearch because we don't support sub-dependencies in get_dependant_params` 😬
| "media_type": tile_format.mediatype, | ||
| }, | ||
| media_type="application/xml", | ||
| ) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👋 in titiler-mosaic now
| ## [Unreleased] | ||
|
|
||
| * update titiler requirements to `>=1.0,<1.1` | ||
|
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TODO: reference all the changes
| api: | ||
| # At the time of writing, rasterio wheels are not available for arm64 arch | ||
| # so we force the image to be built with linux/amd64 | ||
| platform: linux/amd64 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
thanks for recent update of rasterio
|
|
||
|
|
||
| @attr.s | ||
| class STACAPIBackend(BaseBackend): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we now use the base backend from rio-tiler (not cogeo-mosaic)
|
@jverrydt would you be able to test this branch before I merge? The endpoints haven't changed much:
I'll document all the changes, some are documented over https://developmentseed.org/titiler/migrations/v1_migration/#point-endpoint-response-model |
|
@vincentsarago It's ok to merge |
ref: developmentseed/titiler#1279
Huge refactor, will need proper documentation
closes #40