[BUG] When the zoom level is set in `baseMap`, it is automatically converted to an integer.

[YongGoose]

Description

If you set minZoom or 'maxZoom' to a float value like 2.5, it will automatically be converted to 2.

{
  minzoom: 2.5,
  maxzoom: 20,
  sql:
    "SELECT id, tags, geom " +
    "FROM osm_boundary " +
    "WHERE tags ->> 'admin_level' = '2' " +
    "AND tags ->> 'maritime' = 'yes';",
}

Environment

• Browser/Version: chrome
• Version: 0.8.1

Expected vs Actual Behavior

• Expected: Setting minZoom to 2.5 should apply it when the zoom level is 2.5 or higher.
• Actual: However, it actually applies when minZoom is 2.0 or higher.

Attachments (Optional)

Maritime boundaries should be applied from zoom level 2.5 and above, but they can be seen applying from 2.0.

Expected

Actual

[YongGoose]

@bchapuis

If this is a bug, it could be a sub-issue of the below issue.

• Basemap improvements #927

[bchapuis]

I believe the java object uses an integer instead of a double for the zoom level. We can indeed change this as it better corresponds to the maplibre style specification.

[YongGoose]

I believe the java object uses an integer instead of a double for the zoom level. We can indeed change this as it better corresponds to the maplibre style specification.

I believe these two classes are definitely related!

• incubator-baremaps/baremaps-maplibre/src/main/java/org/apache/baremaps/maplibre/style/StyleSource.java

  Line 43 in 28e5bb4

  private Integer minzoom;

• incubator-baremaps/baremaps-maplibre/src/main/java/org/apache/baremaps/maplibre/style/StyleLayer.java

  Line 53 in 28e5bb4

  private Integer minzoom;

If this issue is not resolved by next week, I will take care of it myself.

PS. I am not a member of the Apache Baremaps repository, so I don't have the permissions to register this as a sub-issue. Would you mind handling this? 😅

I think it would be appropriate for these two issues to be registered as sub-issues of the #927.

• [BUG] When the zoom level is set in baseMap, it is automatically converted to an integer. #947
• [FEATURE] Display few major cities #949

[bchapuis]

Thanks for the clarification. I attached it as a sub issue of #927.