update titiler #143
update titiler #143
Conversation
| @@ -30,6 +30,8 @@ | |||
| colormap_dependency=PCColorMapParams, | |||
| reader_dependency=ReaderParams, | |||
| router_prefix=get_settings().item_endpoint_prefix, | |||
| # We remove the titiler default `/map` viewer | |||
| add_viewer=False, | |||
There was a problem hiding this comment.
in titiler 0.8 we added a /map viewer which is added by default with the Tiler Factories
| @@ -39,13 +39,16 @@ def __init__(self, request: Request): | |||
| reader_dependency=ReaderParams, | |||
| router_prefix=get_settings().mosaic_endpoint_prefix, | |||
| backend_dependency=BackendParams, | |||
| add_map_viewer=False, | |||
| add_statistics=False, | |||
| add_mosaic_list=False, | |||
There was a problem hiding this comment.
those are already set to False in the TileFactory so it's mostly informative
https://github.com/stac-utils/titiler-pgstac/blob/master/titiler/pgstac/factory.py#L111-L116
| ) | ||
|
|
||
|
|
||
| @pgstac_mosaic_factory.router.get( | ||
| "/info", response_model=MosaicInfo, response_class=ORJSONResponse | ||
| ) | ||
| def map( | ||
| def mosaic_info( |
There was a problem hiding this comment.
renamed for consistency
| h = asset_info.extra_fields["file:header_size"] | ||
| info["env"] = {"GDAL_INGESTED_BYTES_AT_OPEN": h} | ||
|
|
||
| return info |
There was a problem hiding this comment.
we now return a TypedDict with the URL and a dictionary used to set some GDAL env (if available)
pccommon/setup.py
Outdated
| "fastapi @ git+https://github.com/mmcfarland/fastapi/@982e7caf086bffeace8554da6d69e5f3082f14a3#egg=fastapi", | ||
| "starlette>=0.21.0,<0.22.0", | ||
| "fastapi>=0.87", | ||
| "starlette>=0.22.0,<0.23.0", |
There was a problem hiding this comment.
@mmcfarland is it ok to use starlette 0.22
vincentsarago
force-pushed
the
newTiTilerVersion
branch
from
cb430c2
to
ba72679
Compare
|
Due to the breaking expression changes, we're going to wait to evaluate this after the next release (end of Jan). |
|
Thanks @vincentsarago , I'm ready to merge this. Could you resolve the conflict (I can't push to the fork)? The same dependency upgrades have merged after this PR was created. |
|
conflict resolved, I'll let you go ahead an merge |
This pr does:
pctiler/pctiler/reader_cog.py)Frontend changes
asset_expression(Mosaic and Item)b(e.gb1) (Mosaic and Item) (ref: https://github.com/cogeotiff/rio-tiler/blob/main/docs/src/v4_migration.md#band-names){asset}_b{band_name}(e.gred_b1/green_b1) (Mosaic and Item) (ref: https://github.com/cogeotiff/rio-tiler/blob/main/docs/src/v4_migration.md#multibasereader-expressions)asset_as_bandoption to force expression to be in form of{asset}(e.gred/green) (Mosaic and Item);(previously,was accepted) (Mosaic and Item)algorithmoptionsRef: https://github.com/stac-utils/titiler-pgstac/blob/master/CHANGES.md?plain=1#L55-L63