Tomas Pohanka

Hi, is there still a problem?

Could be similar to https://github.com/openmaptiles/openmaptiles/issues/1431.

Hi, I understand that this is an old issue (and another even older https://github.com/openmaptiles/openmaptiles/issues/761). We should include peak points with a name and without elevation, e.g. as it is now...

Hi, thanks for pointing this out. You are correct, that `hairdresser` should be part of the `shop` class. The reason that not all shops are in `class=shop` is for grouping...

Thanks for pointing this out. [Smoothness](https://wiki.openstreetmap.org/wiki/Key:smoothness) could be considered for zoom level 15.

Thanks! Maybe one improvement for `valley` should be `LineLabel` function (example in `water_name` layer). Some of the valleys could be very short even for zoom 13 e.g. https://www.openstreetmap.org/way/451566241. https://github.com/openmaptiles/openmaptiles/blob/6c31841f4674f15e15afde346a060cf7c22e6cdd/layers/water_name/water_name.sql#L34

Adding the `LineLabel` will be beneficial. Also please consider excluding the `valley` without `name` (or how you would like to style just the line?). Could you please update the style...

Thanks a lot! The performance test will not pass as it uses cache which uses spritezero.

Could you please share the final mbtiles? The small mbtiles do not necessarily mean that is incomplete. Which MAX_ZOOM do you set up in `.env` file? https://github.com/openmaptiles/openmaptiles/blob/b3d67ed5b327c9059aeea0b3304772c6b4c8c7e9/.env#L23

Hi, was it helpful?