Skip to content

Feature/tunnel support#4

Merged
russellporter merged 2 commits intorussellporter:masterfrom
pranet:feature/tunnel-support
Mar 5, 2026
Merged

Feature/tunnel support#4
russellporter merged 2 commits intorussellporter:masterfrom
pranet:feature/tunnel-support

Conversation

@pranet
Copy link
Copy Markdown
Contributor

@pranet pranet commented Feb 28, 2026

Add tunnel property to RunProperties and LiftProperties
Adds support for the OSM tunnel tag on piste and aerialway/railway ways,
which indicates the feature passes through a tunnel rather than over a mountain.
True for any tunnel value (yes, avalanche_protector, building_passage, etc),
null if untagged.

See: https://wiki.openstreetmap.org/wiki/Key:tunnel

Co-Authored-By: Claude Sonnet 4.6 noreply@anthropic.com

pranet and others added 2 commits February 27, 2026 19:00
Adds support for the OSM tunnel tag on piste and aerialway/railway ways,
which indicates the feature passes through a tunnel rather than over a mountain.
True for any tunnel value (yes, avalanche_protector, building_passage, etc),
null if untagged.

See: https://wiki.openstreetmap.org/wiki/Key:tunnel

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Ensures dist/ is built when installing from a git URL (e.g. for
cross-repo development before publishing to npm).

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@russellporter
Copy link
Copy Markdown
Owner

@pranet can you also open a PR on the openskidata-processor with the changes to add the tunnel property?

If you want to additionally adjust the map rendering, the property will also need to be added to on the MapboxGLRunProperties / MapboxGLLiftProperties types in that repo.

For the rendering, the map style is not open source on github but you can modify the style (https://tiles.openskimap.org/styles/terrain_v2.json) using https://maputnik.github.io/ attach your update and I can apply the changes manually

@pranet
Copy link
Copy Markdown
Contributor Author

pranet commented Mar 5, 2026

Yep I was able to test out changes to the openskidata-processor and openskimap.org repositories to verify tunnel info gets correctly propagated and displayed in the sidecard(without affecting map rendering for now) . Let me send the PRs shortly.

@pranet pranet closed this Mar 5, 2026
@pranet pranet reopened this Mar 5, 2026
@pranet
Copy link
Copy Markdown
Contributor Author

pranet commented Mar 5, 2026

@russellporter russellporter/openskidata-processor#40 is the processor PR. It does have a temporary change to package.json which I'll be able to remove once this openskidata-format PR is merged and new npm package is available.

@pranet
Copy link
Copy Markdown
Contributor Author

pranet commented Mar 5, 2026

@russellporter russellporter/openskimap.org#174 is one of the UI PRs. Same temp change dependency as the processor one.

@russellporter russellporter merged commit b957be7 into russellporter:master Mar 5, 2026
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