Repository navigation
fix: prevent operator tag from becoming rest area POI name - #5
Merged
Merged
Conversation
Rest area features in OSM often have no `name` or `brand` tag, only an `operator` tag (e.g. "Florida Department of Transportation"). The lua config fell back to `operator` for the `name` field, causing rest areas to display government agency names instead of "Rest Area". Two fixes: - derive.sql: when a restArea POI has no OSM name/brand tags, force name and display_name to "Rest Area" regardless of operator fallback - openinterstate.lua: remove operator fallback from name/display_name (takes effect on next PBF import; raw tags still stored in jsonb) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
operatortag when nonameorbrandtag existsnameorbrandtags (checked viatags_json), force name/display_name to "Rest Area" — this fixes the current data without requiring a PBF re-importoperatorfallback from name/display_name fields — prevents the issue at the source on next PBF importAffected exits
19 Florida rest area POIs across I-4, I-75, I-95, I-275 were showing "Florida Department of Transportation" as their name. After this fix they'll correctly show "Rest Area".
Test plan
🤖 Generated with Claude Code