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

climbingTiles: add missing routes #935

Merged
merged 1 commit into from
Feb 5, 2025
Merged

climbingTiles: add missing routes #935

merged 1 commit into from
Feb 5, 2025

Conversation

zbycz
Copy link
Owner

@zbycz zbycz commented Feb 5, 2025

With ._ we have on preview:

Starting...
Overpass elements: 201367
Records: 41737
SQL Query length: 14531069 chars
Done.
Duration: 84808 ms

without it it was:
29.1.2025:
Fetching data from Overpass...
Overpass elements: 172826
Records: 23396
SQL Query length: 8243397 chars
Done.
Duration: 71964 ms
image

Copy link

vercel bot commented Feb 5, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
osmapp ✅ Ready (Inspect) Visit Preview Feb 5, 2025 5:12am

@zbycz zbycz changed the title tiles-fix climbingTiles: add missing routes Feb 5, 2025
Repository owner deleted a comment from github-actions bot Feb 5, 2025
Repository owner deleted a comment from github-actions bot Feb 5, 2025
@zbycz zbycz merged commit 59fc41a into master Feb 5, 2025
6 of 7 checks passed
@zbycz zbycz deleted the tiles-fix branch February 5, 2025 05:16
@@ -40,7 +40,7 @@ const prepareGeojson = (

const fetchFromOverpass = async () => {
// takes about 42 secs, 25MB
const query = `[out:json][timeout:80];(nwr["climbing"];nwr["sport"="climbing"];);>>;out qt;`;
const query = `[out:json][timeout:80];(nwr["climbing"];nwr["sport"="climbing"];);(._;>>;);out qt;`;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

black magic :D

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.

2 participants