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 power=plant to area creation rules #609

Open
joconnor-ecaa opened this issue Jan 12, 2021 · 4 comments · May be fixed by #610
Open

Add power=plant to area creation rules #609

joconnor-ecaa opened this issue Jan 12, 2021 · 4 comments · May be fixed by #610

Comments

@joconnor-ecaa
Copy link

joconnor-ecaa commented Jan 12, 2021

Posting an issue as instructed by https://wiki.openstreetmap.org/wiki/Overpass_API/Areas. My use case involves querying for objects within the area of a power plant way.

The area creation rules as specified here include power=station, power=generator, power=sub_station and power=transformer but no power=plant. Most power plants have landuse=industrial but it if the plant is left without a name, no area is created. It also seems that the landuse=industrial tag may be disputed for power plants.

Happy to PR myself if this change is acceptable.

@joconnor-ecaa
Copy link
Author

joconnor-ecaa commented Jan 12, 2021

On closer inspection, everything requires a name. I'm guessing this prevents computation over spurious objects?

@joconnor-ecaa joconnor-ecaa linked a pull request Jan 14, 2021 that will close this issue
@mmd-osm
Copy link
Contributor

mmd-osm commented Jan 14, 2021

Fyi: the concept of using hard coded rules to define areas will quite likely disappear later this year, meaning that you will be able to treat every closed object as an area.

@isabellasoldner
Copy link

Would it be possible to add power=plant as a interim fix until the code is refactored?

@mmd-osm
Copy link
Contributor

mmd-osm commented Jan 15, 2021

I’d say it’s somewhat unlikely to see this in production in the near future. Probably you need to look into setting up your own overpass instance, or use some alternative approaches.

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

Successfully merging a pull request may close this issue.

4 participants