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

matcher #399

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

matcher #399

wants to merge 14 commits into from

Conversation

wipfli
Copy link
Collaborator

@wipfli wipfli commented Feb 25, 2025

Implements the ideas from #396

I converted the landuse layer completely.

The pois layer is not finished yet.

@wipfli
Copy link
Collaborator Author

wipfli commented Feb 27, 2025

The performance sees not to be affected by this pull request. I run the landuse layer on the planet.osm.pbf with

java -Xmx30g -jar target/protomaps-basemap-HEAD-with-deps.jar \
  --nodemap-type=array \
  --osm_path=/home/oliver/osm/planet-latest.osm.pbf \
  --output=data/v5.pmtiles --force \
  --maxzoom 15 \
  --layer=landuse \
  --output-layerstats --log_jts_exceptions --download | tee logs.txt

and got:

  • main 30 min 43 sec
  • matcher 30 min 18 sec

@wipfli
Copy link
Collaborator Author

wipfli commented Feb 27, 2025

On @msbarry suggestion I added multi-line string support. You can now write

with("highway", "primary", "secondary")

as

with("""
  highway
  primary
  secondary
""")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant