Modify

Opened 13 years ago

Closed 13 years ago

#7812 closed enhancement (fixed)

warn about missing power=tower/pole within powerlines

Reported by: skyper Owned by: team
Priority: normal Milestone:
Component: Core validator Version: latest
Keywords: power pole tower Cc:

Description

Please, have validator warn about nodes which are part of a power=line/minor_line and do not have a power=tower/pole tag.

Thanks

r5295

Attachments (0)

Change History (9)

comment:1 by Don-vip, 13 years ago

Resolution: fixed
Status: newclosed

In 5300/josm:

fix #7812 - warn about missing power=tower/pole within powerlines

comment:2 by skyper, 13 years ago

This was fast.

Thanks a lot :-)

Last edited 13 years ago by skyper (previous) (diff)

in reply to:  1 ; comment:3 by adjuva@…, 13 years ago

fix #7812 - warn about missing power=tower/pole within powerlines

The idea for this check is very good - too many mappers insert nodes to power lines by accident, when they edit landuses or ways.

But the current implementation creates too many false positives.

You should change the following details:

  • no warnings should be generated for the start and end node of a way
  • no warnings should be generated for nodes of ways with the tags power=line, line=busbar
  • and if you can implement it: no warnings should be generated for nodes which are inside an area with the tags power=station|sub_station|generator

comment:4 by Don-vip, 13 years ago

Resolution: fixed
Status: closedreopened

comment:5 by Don-vip, 13 years ago

Resolution: fixed
Status: reopenedclosed

In 5312/josm:

fix #7812 - validator: power lines test improvements (no warning for start/end, node with power infrastructure tags, node inside power facilities)

in reply to:  3 ; comment:6 by skyper, 13 years ago

Replying to adjuva:

You should change the following details:

  • no warnings should be generated for the start and end node of a way

I often met powerlines which are parted due to different tags or simply because the line splits. I would rather say:

  • do not warn at start/end node if no other way with tag power=line/minor_line is connected.

EDT: I tend to prefer some false possitives than to many misses as you can always ignore hits (with or without list) but it is much harder to find misses.

Last edited 13 years ago by skyper (previous) (diff)

in reply to:  6 ; comment:7 by Don-vip, 13 years ago

Resolution: fixed
Status: closedreopened

Replying to skyper:

Replying to adjuva:
I often met powerlines which are parted due to different tags or simply because the line splits. I would rather say:

  • do not warn at start/end node if no other way with tag power=line/minor_line is connected.

Agreed. I could also completely remove the first/last node check now there is the "is in power facility" check. Is there a case where a power line can start/finish outside of a power station without a propoer power tag on its first/last node ?

in reply to:  7 comment:8 by skyper, 13 years ago

Replying to Don-vip:

Replying to skyper:

Replying to adjuva:
I often met powerlines which are parted due to different tags or simply because the line splits. I would rather say:

  • do not warn at start/end node if no other way with tag power=line/minor_line is connected.

Agreed. I could also completely remove the first/last node check now there is the "is in power facility" check. Is there a case where a power line can start/finish outside of a power station without a proper power tag on its first/last node ?

Do not any real cases, but maybe ask a bigger crowd at talk@ or tagging@.
The only case I know is that the line is not completely mapped so far, but then the warning is correct.

comment:9 by Don-vip, 13 years ago

Resolution: fixed
Status: reopenedclosed

In 5359/josm:

fix #7812 - Validator: warn about missing power=tower/pole within powerlines, even for first and last node of the line.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.