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

Add operator presets for highway=street_lamp (CenterPoint Energy, Puget Sound Energy, and Seattle City Light) #9851

Merged
merged 2 commits into from
Sep 1, 2024

Conversation

CorruptComputer
Copy link
Contributor

@CorruptComputer CorruptComputer commented Aug 28, 2024

CenterPoint Energy owns and operates all of the street lights in my area, having a preset for this would make it easy to add that info. Used this as a reference for the values of that:

"displayName": "CenterPoint Energy",

Discussed this in the OSM Discord, and someone had these values they were using for ones in their area, so I added them as well:
https://github.com/Lumikeiju/openstreetmap/blob/main/presets/seattle_city_light_pole.xml

@Snowysauce Snowysauce added the new category Add support for a new kind of feature label Aug 28, 2024
@Snowysauce
Copy link
Collaborator

I like this idea - so much so that I filed a feature request with iD to make adding street light operator names easier. I don't personally map them, but a quick overpass turbo search of my area showed a couple hundred mapped lights, all without operator details. I can't help but think that's linked to currently having to add the operator via the "tags" section.

@CorruptComputer
Copy link
Contributor Author

Thanks for that suggestion, I've opened a second PR over in the id-tagging-schema repo adding that as well:
openstreetmap/id-tagging-schema#1327

@arch0345 arch0345 merged commit ca6e5b8 into osmlab:main Sep 1, 2024
2 checks passed
@CorruptComputer CorruptComputer deleted the add-street-lamp branch September 1, 2024 02:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new category Add support for a new kind of feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants