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

Use tile dimensions that are defined by the TMS & deprecate or remove tile_size config option #49

Open
robyngit opened this issue Jan 14, 2025 · 0 comments
Labels
bug Something isn't working good first issue Good for newcomers

Comments

@robyngit
Copy link
Member

The workflow allows configuration of custom tile sizes, but problematically, the tile dimensions (height and width) are inherently part of each Tile Matrix Set (TMS) definition. We should therefore remove or deprecate the current tile_size option in viz-staging and instead retrieve the height and width values directly from the TMS using the Morecantile library. This issue is in the viz-workflow repo because tile_size is used in both viz-staging and viz-raster.

In the future, if we need custom tile sizes, we should instead define a custom TMS. We could consider keeping tile_size as a private option for developers, since running the workflow with smaller tiles is useful for fast testing & debugging.

@robyngit robyngit added good first issue Good for newcomers bug Something isn't working labels Jan 14, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working good first issue Good for newcomers
Projects
Status: No status
Development

No branches or pull requests

1 participant