ConservationMetrics / mapgl-tile-renderer

Headless Node.js Maplibre-GL renderer for generating MBTiles with styled raster tiles.
MIT License
12 stars 3 forks source link

Add action to build tiles from input #48

Closed luandro closed 8 months ago

luandro commented 8 months ago

Goal

Add ability to build tiles from inputs in Actions tab in Github.

rudokemper commented 8 months ago

Hey @luandro can you describe a bit more the intended use case for this?

luandro commented 8 months ago

Might not be very relevant once the UI is working. It's just a way to build tiles from the Action tab, without having to use the manifest like in the template, in case somebody forks for example and wants to build without touching code.

image

rudokemper commented 8 months ago

That's fine, I think this repo can support different use cases (CLI, web app, build pipelines). Could you add to the documentation in the readme, though? That way others can know about this possibility.

rudokemper commented 8 months ago

@luandro I made a few changes to be consistent in the usage of flag names (and when they are optional or not), and to add a couple of new flags that were introduced (tiletype, ratio, openstreetmap). The most impactful change is that I made filename (formerly tile_name) optional in the build script, since there is already a default fallback value provided for that in the code. Let me know if you agree with my changes.

luandro commented 8 months ago

There's a max of 10 input fields. Would have to remove one (if i'm counting it right).

rudokemper commented 8 months ago

Ah, I see. Then I'd remove tiletype and radio.

luandro commented 8 months ago

Looking good now!