-
Notifications
You must be signed in to change notification settings - Fork 5
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
Release changes #178
Release changes #178
Conversation
Collection search
fix bad link
Update deploying_to_existing_environments.md
…da environment to agree to pay all S3 transfer costs
use titiler custom JSONResponse to handle NaN values
@@ -1,11 +1,11 @@ | |||
FROM lambci/lambda:build-python3.8 | |||
FROM public.ecr.aws/sam/build-python3.9:latest |
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.
FROM public.ecr.aws/sam/build-python3.9:latest | |
FROM --platform=linux/amd64 public.ecr.aws/sam/build-python3.9:latest |
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.
i've had issues with deploying from the new mac, this solves it
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.
might as well do it for all the Dockerfiles
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.
wait, not just mac, but also from github actions 🤔
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.
Even with export DOCKER_DEFAULT_PLATFORM=linux/amd64
? This has been deploying from github actions as is. I'm reading up, though, to see if we should take out a branch from develop to make the change.
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.
Isn't github actions running on ubuntu so shouldn't need --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.
thought so too, maybe i am misremembering. but it's always a good idea to add it whereever.
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.
I agree with that. I've run into so many problems with this when running locally on my mac.
Change db search path
router=APIRouter(route_class=LoggerRouteHandler), | ||
) | ||
app.include_router(stac.router, tags=["Items"], prefix="/stac") | ||
|
||
cog = TilerFactory( | ||
router_prefix="/cog", | ||
optional_headers=optional_headers, | ||
gdal_config=settings.get_gdal_config(), | ||
environment_dependency=settings.get_gdal_config, |
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.
So environment_dependency
is a callable and not a dict
anymore in titiler. That was one of the breaking changes when gdal_config
was removed:
Here are the change notes:
https://github.com/developmentseed/titiler/blob/main/CHANGES.md#070-2022-06-08
Export AWS Access Credentials If Needed
Do not advertise experimental endpoint
…s-cfg-in-function Put raster assume role config in lambda construct
Changes
Dashboard-facing changes
b1
,b2
, etc. Previous versions of titiler used numeric band names for some COGs. This is visible in the/cog/statistics
andcog/info
resultsBackend updates
/collection-id-search
endpoint searches for all items meeting filter criteria and returns unique ids (may be updated/replaced by stac api aggregation extension). Long term support is not committed for this endpoint.