Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#20270 closed enhancement (fixed)

[PATCH] Add `man_made=utility_pole` to presets

Reported by: taylor.smock Owned by: team
Priority: normal Milestone: 21.10
Component: Internal preset Version:
Keywords: pole Cc:

Description (last modified by taylor.smock)

Current usage is >20,000 (mostly in the past year -- see https://taginfo.openstreetmap.org/tags/man_made=utility_pole#chronology ).

Current patch notes:

  • Reuses the icon for power=pole in presets
  • Adds two new chunks
    • utilities for presets that share utility keys
    • pole for pole-like objects (height, material, and ref_operator)
  • Adds the utility_pole preset

Note: Patch is now being tracked at https://gitlab.com/smocktaylor/josm/-/merge_requests/4

Attachments (25)

20270.patch (5.3 KB ) - added by taylor.smock 3 years ago.
Add man_made=utility_pole to presets
pole.svg (9.3 KB ) - added by taylor.smock 3 years ago.
Possible pole icon (same as powerpole, just with a white halo)
pole.2.svg (9.3 KB ) - added by taylor.smock 3 years ago.
Pole, missing right, left upraised
pole.3.svg (9.3 KB ) - added by taylor.smock 3 years ago.
Pole, missing right, left down
pole.4.svg (9.3 KB ) - added by taylor.smock 3 years ago.
Pole, both raised
pole.5.svg (9.3 KB ) - added by taylor.smock 3 years ago.
Pole, missing left, right upraised
pole.6.svg (9.3 KB ) - added by taylor.smock 3 years ago.
Pole, missing left, right down
pole_little_upper.svg (7.9 KB ) - added by skyper 3 years ago.
pole with little upper more inside
pole_upper.svg (7.9 KB ) - added by skyper 3 years ago.
pole with upper more inside
pole_telecom.svg (7.0 KB ) - added by skyper 3 years ago.
green casing
pole_power.svg (7.0 KB ) - added by skyper 3 years ago.
yellow casing
pole_light.svg (7.0 KB ) - added by skyper 3 years ago.
orange casing
pole_light_yellow.svg (7.0 KB ) - added by skyper 3 years ago.
lighter yellow for street lights
poles_test.png (36.8 KB ) - added by francois.lacombe 3 years ago.
Thank you Gaben for this style, very useful. I've tested several combinations and this one seems to be acceptable.
pole_street_light.svg (1.8 KB ) - added by francois.lacombe 3 years ago.
A slight change in geometry for street_light utility poles
20270.2.patch (12.8 KB ) - added by taylor.smock 3 years ago.
Update to current JOSM, add preset links
20270.3.patch (50.9 KB ) - added by taylor.smock 3 years ago.
Add icons for specific utilities
20270.4.patch (7.3 KB ) - added by taylor.smock 3 years ago.
Remove svg icons whose only difference was halo, replace with mapcss psuedo halo
Screen Shot 2021-09-30 at 8.50.32 AM.png (4.1 KB ) - added by taylor.smock 3 years ago.
Left to right: no utility, utility=gas, utility=water, utility=television, utility=telecom, utility=power, utility=waste, utility=telecom;waste, utility=street_lighting
Screen Shot 2021-09-30 at 8.59.03 AM.png (4.2 KB ) - added by taylor.smock 3 years ago.
Left to right: no utility, utility=gas, utility=water, utility=television, utility=telecom, utility=power, utility=waste, utility=telecom;waste, utility=street_lighting
20270.5.patch (7.4 KB ) - added by taylor.smock 3 years ago.
Add halos for some missing utility types that also appear in substance
20270.6.patch (7.5 KB ) - added by taylor.smock 3 years ago.
Drop opacity, add text= to preset_links
20270.7.patch (7.4 KB ) - added by taylor.smock 3 years ago.
Change preset_link text from Similar but different tags to Similar preset. (AKA: New string for translation)
josm_20270_8.patch (7.4 KB ) - added by skyper 3 years ago.
version 8: mapcss: new color for sewerage; remove duplicate gas; support only single value for telecom/television
20270.size_12.patch (7.4 KB ) - added by taylor.smock 3 years ago.
Same as attachment:josm_20270_8.patch, but halos are size 12 (static)

Download all attachments as: .zip

Change History (90)

by taylor.smock, 3 years ago

Attachment: 20270.patch added

Add man_made=utility_pole to presets

comment:1 by taylor.smock, 3 years ago

For the different pole icon, I'm thinking one of two different options:
1) Modify the power pole icon to have a different surrounding color (probably white)
2) Use https://github.com/mapillary/mapillary_sprite_source/blob/master/package_objects/object--support--utility-pole.svg with some modifications (MIT licensed)

by taylor.smock, 3 years ago

Attachment: pole.svg added

Possible pole icon (same as powerpole, just with a white halo)

comment:2 by skyper, 3 years ago

Component: CoreInternal preset

comment:3 by taylor.smock, 3 years ago

4 week keep-alive ping (oops).

Which pole icon would be preferred? This is going to affect the name of the icon (and hence the final patch). If I use the one from Mapillary, I'm going to keep the filename as object--support--utility-pole.svg, while if I use the modified powerpole.svg icon, I would use the generic pole.svg.

comment:4 by taylor.smock, 3 years ago

Summary: Add `man_made=utility_pole` to presets[RFC PATCH] Add `man_made=utility_pole` to presets

comment:5 by francois.lacombe, 3 years ago

Hi all

This value is going through a formal proposal which is currently in RFC state
https://wiki.openstreetmap.org/wiki/Proposed_features/Utility_poles_proposal

Shouldn't we wait vote to occur as to not have to change the preset right after?

Feel free to comment the proposal is you have input for it
That's great to see useful icon for that as well, thank you

All the best

Last edited 3 years ago by francois.lacombe (previous) (diff)

comment:6 by taylor.smock, 3 years ago

I was not aware that there was a proposal for it.

It does, however, have a non-proposal wiki page ( https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dutility_pole ), which indicates that the tag status is in use.

Also, as far as I can tell, the proposal is ~7 years newer than the Tag:man_made=utility_pole page, but better and more well-defined documentation is always welcome.

comment:7 by francois.lacombe, 3 years ago

Yeah I'm aware of the Tag page and it indeed exists since many years.

The proposal is here to encourage a formal discussion about how this value may be useful and with which tags it should be combined.
There may be kind of confusion between power=pole and more general man_made=utility_pole.

This proposal won't deprecate man_made=utility_pole but encourage it actually so it won't cause any harm to add a preset to JOSM prior to vote
Some problems could occur with combined tags and power=pole duplicity and it could impact on preset and validation rules.
As power=pole isn't a really good power value, the proposal will deprecate telecom=pole prior to its establishment to prevent any similar problem as well

comment:8 by taylor.smock, 3 years ago

Four month I forgot about this ping.

I probably need to update the patch, but I'd like to know from the core devs if they want a utility_pole preset.

The two images side by side (48px). Left is attachment:pole.svg and right is from the mapillary_sprite_source (MIT licensed).
Possible pole icon (same as powerpole, just with a white halo) or https://raw.githubusercontent.com/mapillary/mapillary_sprite_source/master/package_objects/object--support--utility-pole.svg

comment:9 by skyper, 3 years ago

Comparing the power pole and utility_pole, source:trunk/resources/images/presets/power/pole.svg Possible pole icon (same as powerpole, just with a white halo), I think the new icon should look a little more different. How about only one (or on one side only a small) horizontal bar and/or little vertical lines on top of the horizontal bar instead of below.

comment:10 by gaben, 3 years ago

I think it's not ready to be included in the core JOSM presets. The tagging doesn't seem established for me.

Only 17.5% of the man_made=utility_poles use utility= secondary tag, and 80% of it is utility=telecom.

Correct me if I'm wrong, but also duplicates tags; power pole with lighting tagged as

  • man_made=utility_pole
  • utility=power;street_lighting
  • highway=street_lamp
  • power=pole

is same as

  • highway=street_lamp
  • power=pole.
Last edited 3 years ago by gaben (previous) (diff)

in reply to:  10 comment:11 by taylor.smock, 3 years ago

Replying to gaben:

Only 17.5% of the man_made=utility_poles use utility= secondary tag, and 80% of it is utility=telecom.

I don't know about you, but I cannot tell the difference between a pole used by a telecom and a powerpole when randomly mapping poles. Once you have connected it back to a substation/power plant, it becomes much easier to determine.

AFAIK, there are only two major uses for utility poles: power, and communication. (power, television, telecom). So, with power=pole having been widespread already, it isn't terribly surprising that most utility tags for man_made=utility_pole is telecom.

Correct me if I'm wrong, but also duplicates tags; power pole with lighting tagged as

  • man_made=utility_pole
  • utility=power;street_lighting
  • highway=street_lamp
  • power=pole

is same as

  • highway=street_lamp
  • power=pole.

Reading the proposed spec, if the utility pole was primarily installed as a power pole,
man_made=utility_pole + utility=power and (mandatory) power=pole. I suspect power=pole is mostly mandatory for compatibility reasons though.
highway=street_lamp

With all that being said, I would much rather have people map a more generic pole when they cannot reliably determine the usage, and then allow applications like StreetComplete to fill in the missing information, instead of the mapper having to assume power=pole or communication=pole (or something else).

by taylor.smock, 3 years ago

Attachment: pole.2.svg added

Pole, missing right, left upraised

by taylor.smock, 3 years ago

Attachment: pole.3.svg added

Pole, missing right, left down

by taylor.smock, 3 years ago

Attachment: pole.4.svg added

Pole, both raised

by taylor.smock, 3 years ago

Attachment: pole.5.svg added

Pole, missing left, right upraised

by taylor.smock, 3 years ago

Attachment: pole.6.svg added

Pole, missing left, right down

in reply to:  9 comment:12 by taylor.smock, 3 years ago

Replying to skyper:

Comparing the power pole and utility_pole, source:trunk/resources/images/presets/power/pole.svg Possible pole icon (same as powerpole, just with a white halo), I think the new icon should look a little more different. How about only one (or on one side only a small) horizontal bar and/or little vertical lines on top of the horizontal bar instead of below.

Pole, missing right, left upraised Pole, both raised Pole, missing left, right upraised

Pole, missing right, left down Possible pole icon (same as powerpole, just with a white halo) Pole, missing left, right down

Original power pole:
source:trunk/resources/images/presets/power/pole.svg

To me, the ones that are missing on side appear to be lopsided.

comment:13 by taylor.smock, 3 years ago

Description: modified (diff)

comment:14 by gaben, 3 years ago

The only thing I don't like is mixed tags "for compatibility". Make it clear and suggest replacing the current ones, but it would need community approval...

One thing no one mentioned yet, how will the line between the poles tagged? Because after all, the cables determine everything in this regard.

comment:15 by francois.lacombe, 3 years ago

Dear all,

The proposal reviewing man_made=utility_pole has been accepted.
Wikipage is now up to date
https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dutility_pole

Indeed, power=pole is still the correct tagging for power poles and we still lack a proper tag to tag telecom lines between poles.
Why can't we review later telecom=line?
https://wiki.openstreetmap.org/wiki/Telecoms is also up to date

All the best

comment:16 by skyper, 3 years ago

Two more possible icons: pole with upper more inside,  pole with little upper more inside

by skyper, 3 years ago

Attachment: pole_little_upper.svg added

pole with little upper more inside

by skyper, 3 years ago

Attachment: pole_upper.svg added

pole with upper more inside

comment:17 by Don-vip, 3 years ago

Keywords: pole added

comment:18 by skyper, 3 years ago

I would add presets links between the two pole presets, e.g. utility pole -> power pole and power pole -> utility pole

comment:19 by francois.lacombe, 3 years ago

Hi Skyper

Regarding icons, I think that power=pole and man_made=utility_pole should have the same shape.
Background or casing colors should adapt regarding utility=* value.

  • utility=power (including implied by power=pole) => yellow (existing)
  • utility=telecom => green
  • Unknown utility => grey (or no casing)
  • utility=street_lighting => to be defined (yellow ?)

It's already successfully done for man_made=street_cabinet
Should we design one and each icon for every color or can josm automatically adapt background color?

Agree on links between power=pole and man_made=utility_pole.

in reply to:  19 ; comment:20 by skyper, 3 years ago

Replying to francois.lacombe:

Regarding icons, I think that power=pole and man_made=utility_pole should have the same shape.
Background or casing colors should adapt regarding utility=* value.

Fine, let's use casing

  • utility=telecom => green

Ok

  • Unknown utility => grey (or no casing)

Atm, it is white, which is invisible with white background.

  • utility=street_lighting => to be defined (yellow ?)

yellow is used for power, how about a light blue or light orange?

It's already successfully done for man_made=street_cabinet
Should we design one and each icon for every color or can josm automatically adapt background color?

I am not an expert regarding icons. Think we need an icon per color and set them in mappaint. Have similar issue with dark mode (laf) and solid icons but without a solution, so far.

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

in reply to:  20 ; comment:21 by francois.lacombe, 3 years ago

Replying to skyper:

Atm, it is white, which is invisible with white background.

I think no casing at all is better

  • utility=street_lighting => to be defined (yellow ?)

yellow is used for power, how about a light blue or light orange?

Light orange, that's good

I am not an expert regarding icons. Think we need an icon per color and set them in mappaint. Have similar issue with dark mode (laf) and solid icons but without a solution, so far.

That's understandable.
I'm ok to design required casing.
Can you tell me which shape was selected among ones proposed upside please?

by skyper, 3 years ago

Attachment: pole_telecom.svg added

green casing

by skyper, 3 years ago

Attachment: pole_power.svg added

yellow casing

by skyper, 3 years ago

Attachment: pole_light.svg added

orange casing

in reply to:  21 comment:22 by skyper, 3 years ago

Replying to francois.lacombe:

Replying to skyper:

Atm, it is white, which is invisible with white background.

I think no casing at all is better

Mmh, casing works better with dark background in cases of dark imagery or non and additionally in presets with dark modes (laf).

  • utility=street_lighting => to be defined (yellow ?)

yellow is used for power, how about a light blue or light orange?

Light orange, that's good

Ok. Looking at the casing color for power again, it is not completely yellow but has some red in it. Now I am puzzled what to do, see below.

I am not an expert regarding icons. Think we need an icon per color and set them in mappaint. Have similar issue with dark mode (laf) and solid icons but without a solution, so far.

That's understandable.
I'm ok to design required casing.
Can you tell me which shape was selected among ones proposed upside please?

I think, no change in shape is the solution. E.g. using the original power one.
Creating the icons should be easy but if we change colors of existing ones, all icons need to be adjusted.
Regarding casing color, I would prefer little lighter colors in general. Either the casing for power should be changed to complete yellow (for all power icons) or "light" gets the complete yellow which might be less work. In my eyes, the green in use for "telecom" is too dark, too.

My proposal:
Possible pole icon (same as powerpole, just with a white halo) general utility_pole (white casing) also for preset
yellow casing power pole also for preset, former source:trunk/resources/images/presets/power/pole.svg
orange casing light pole
green casing telecom pole

comment:23 by francois.lacombe, 3 years ago

That's good, thank you. I think we're almost done here.

Regarding telecom green, I'd suggest a little darker: #00b70e
Regarding power, we should keep the same yellow as existing, as it matches with many other power icons. As light pole should be distinguishable, what about #ff7d7d (medium red) for utility=street_lighting?

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

by skyper, 3 years ago

Attachment: pole_light_yellow.svg added

lighter yellow for street lights

comment:24 by skyper, 3 years ago

I chose light colors for the casing on purpose, to have more contrast within the icons.
I think red is not recognizable a pole for street lights, therefor I would stick with yellow, maybe, even a little bit lighter than the one I proposed earlier like #ffff4cff

power ? light ? power
as now
light ?
orange casing yellow casing source:trunk/resources/images/presets/power/pole.svg lighter yellow for street lights
ffb233ff ffff33ff ffcc00ff ffff4cff

comment:25 by francois.lacombe, 3 years ago

I think we should be consistent with existing icons at first, the current green for telecom street cabinet looks more appropriate don't you?

Regarding power/light, the two at the right sounds good to me, difference between light orange and solid yellow is enough.

  • ffcc00ff for power
  • ffff4cff for light

comment:26 by skyper, 3 years ago

Sure, always the same colors per infrastructure. I simply would change all telecom icons to a lighter green. Would have done the same if the color for power would change.

comment:27 by francois.lacombe, 3 years ago

I find the proposed green more flashy than the current one.
May someone will have further comment about it?

Power color change isn't necessary I think.

in reply to:  27 comment:28 by gaben, 3 years ago

Load the icons and see which is best. You can use the Styles/IconTester.

by francois.lacombe, 3 years ago

Attachment: poles_test.png added

Thank you Gaben for this style, very useful. I've tested several combinations and this one seems to be acceptable.

by francois.lacombe, 3 years ago

Attachment: pole_street_light.svg added

A slight change in geometry for street_light utility poles

comment:29 by francois.lacombe, 3 years ago

Finally I find #00a306 more appropriate for telecom pole, depending on your feeling here for pole 4739.

Thank you Gaben for this style, very useful. I've tested several combinations and this one seems to be acceptable.

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

comment:30 by skyper, 3 years ago

Summary: [RFC PATCH] Add `man_made=utility_pole` to presets[PATCH] Add `man_made=utility_pole` to presets

comment:31 by Don-vip, 3 years ago

Milestone: 21.09

comment:32 by Don-vip, 3 years ago

Not sure to understand the scope of this ticket. The patch only adds a single icon pole.svg, and there's a lot of other icons being discussed.

comment:33 by skyper, 3 years ago

I think the patch is not read for commit, yet.
Concerning the preset:

  • ref=* and operator=* should be added as optional keys (instead of removing them from power=pole).
  • preset_links between the similar but different presets should be added.
  • is heating as value for utility=* ok, or should it be in sync with substance=*? (#21358).

The different icons are for the mappaint style but this part is missing completely, so far.

Have we decided which icons to use?

  • Possible pole icon (same as powerpole, just with a white halo) as general icon
  • A slight change in geometry for street_light utility poles for utility=street_lighting
  • source:trunk/resources/images/presets/power/pole.svg for utility=power
  • general icon but with green halo for utility=telecom, utility=television
  • general icon but nautical water blue (source:trunk/resources/images/presets/nautical/water.svg) halo for utility=water
  • general icon but yellow halo for utility=gas
  • general icon but pink halo for utility=waste

What colors for halo should be used for chemical, heating, hydrant, oil and sewerage.

  • in #20530, I used pink (source:trunk/resources/images/presets/landmark/pump_sewage.svg) for substance=sewage which should probably be changed to a darker blue.
  • in #20530, I used quite dark background for substance=oil (source:trunk/resources/images/presets/landmark/pump_oil.svg) which might not work well as halo for the pole icon but I`ll try it out.
Last edited 3 years ago by skyper (previous) (diff)

comment:34 by skyper, 3 years ago

Owner: changed from team to skyper

in reply to:  32 ; comment:35 by taylor.smock, 3 years ago

Replying to Don-vip:

Not sure to understand the scope of this ticket. The patch only adds a single icon pole.svg, and there's a lot of other icons being discussed.

I don't either anymore...

I was kind of waiting for people to decide on the icons before updating it again. I'm kind of bad at making good-looking UI elements.

Replying to skyper:

  • ref=* and operator=* should be added as optional keys (instead of removing them from power=pole).

They are still included in the pole chunk which is included in the power_pole chunk.

  • preset_links between the similar but different presets should be added.

Fair.

  • is heating as value for utility=* ok, or should it be in sync with substance=*? (#21358).

heating is probably ok. At least based off of https://wiki.openstreetmap.org/wiki/Key:utility .

in reply to:  35 ; comment:36 by skyper, 3 years ago

Replying to taylor.smock:

Replying to skyper:

  • ref=* and operator=* should be added as optional keys (instead of removing them from power=pole).

They are still included in the pole chunk which is included in the power_pole chunk.

Sorry, I missed that line, so only preset_links should be added.

by taylor.smock, 3 years ago

Attachment: 20270.2.patch added

Update to current JOSM, add preset links

in reply to:  36 comment:37 by taylor.smock, 3 years ago

Replying to skyper:

Replying to taylor.smock:

They are still included in the pole chunk which is included in the power_pole chunk.

Sorry, I missed that line, so only preset_links should be added.

No worries. Everyone misses stuff. And I just attached an updated patch with preset_links. Icons still need to be changed/updated, but I don't know if everyone is happy with the current icon list, so I'm not touching that (yet).

in reply to:  33 comment:38 by gaben, 3 years ago

Replying to skyper:

  • is heating as value for utility=* ok, or should it be in sync with substance=*? (#21358).

In this case, I think it's fine.

by taylor.smock, 3 years ago

Attachment: 20270.3.patch added

Add icons for specific utilities

comment:39 by skyper, 3 years ago

Owner: changed from skyper to team

Ok, I see Taylor is working on the icons. So just a few comments:

  • We only need plain svg icons, which makes the files a lot smaller plus we have difficulties with text in svgs, so all text needs to be replaced with drawing and font-*, text-* and letter-* should be removed from the files.
  • I like the street_lighting icon but for the rest would use the simple pole icon with different halo. Maybe tiny for the general one and grey for mixed or unknown utility=*
  • The green halo should also be used for the combinations telecom;television and television;telecom.
Last edited 3 years ago by skyper (previous) (diff)

in reply to:  39 comment:40 by taylor.smock, 3 years ago

Replying to skyper:

Ok, I see Taylor is working on the icons. So just a few comments:

I literally just took the icons that were available up above and used those.

  • We only need plain svg icons, which makes the files a lot smaller plus we have difficulties with text in svgs, so all text needs to be replaced with drawing and font-*, text-* and letter-* should be removed from the files.

I'll check that -- I don't think I was deliberately using text items.

  • I like the street_ligthning icon but for the rest would use the simple pole icon with different halo. Maybe tiny for the general one and grey for mixed or unknown utility=*
  • The green halo should also be used for the combinations telecom;television and television;telecom.

by taylor.smock, 3 years ago

Attachment: 20270.4.patch added

Remove svg icons whose only difference was halo, replace with mapcss psuedo halo

by taylor.smock, 3 years ago

Left to right: no utility, utility=gas, utility=water, utility=television, utility=telecom, utility=power, utility=waste, utility=telecom;waste, utility=street_lighting

in reply to:  39 comment:41 by taylor.smock, 3 years ago

Replying to skyper:

Ok, I see Taylor is working on the icons. So just a few comments:

  • We only need plain svg icons, which makes the files a lot smaller plus we have difficulties with text in svgs, so all text needs to be replaced with drawing and font-*, text-* and letter-* should be removed from the files.
  • I like the street_lighting icon but for the rest would use the simple pole icon with different halo. Maybe tiny for the general one and grey for mixed or unknown utility=*
  • The green halo should also be used for the combinations telecom;television and television;telecom.

Left to right: no `utility`, `utility=gas`, `utility=water`, `utility=television`, `utility=telecom`, `utility=power`, `utility=waste`, `utility=telecom;waste`, `utility=street_lighting`
Left to right: no utility, utility=gas, utility=water, utility=television, utility=telecom, utility=power, utility=waste, utility=telecom;waste, utility=street_lighting

Left to right: no `utility`, `utility=gas`, `utility=water`, `utility=television`, `utility=telecom`, `utility=power`, `utility=waste`, `utility=telecom;waste`, `utility=street_lighting`
Same as above, just that no utility has a smaller halo.

Note: There is no way (as of yet) to have a halo/outline around the symbol.

Last edited 3 years ago by taylor.smock (previous) (diff)

by taylor.smock, 3 years ago

Left to right: no utility, utility=gas, utility=water, utility=television, utility=telecom, utility=power, utility=waste, utility=telecom;waste, utility=street_lighting

comment:42 by skyper, 3 years ago

Nice, you used mapcss for the background. In this case, I would remove the background for no utility and use the small background for the mixed.
Regarding color of background, in #20530, I tried to sync the colors for substance=* and power:source=*. For identical/similar values of utility=* it might be useful to use the same colors, like yellow for gas. For telecom we already have a dark green with man_made=street_cabinet.

by taylor.smock, 3 years ago

Attachment: 20270.5.patch added

Add halos for some missing utility types that also appear in substance

in reply to:  42 ; comment:43 by taylor.smock, 3 years ago

Replying to skyper:

Regarding color of background, in #20530, I tried to sync the colors for substance=* and power:source=*. For identical/similar values of utility=* it might be useful to use the same colors, like yellow for gas. For telecom we already have a dark green with man_made=street_cabinet.

They were synced already. Except for the telecom street_cabinet color. Something to note is that I added a transparency to the halos, so the apparent color does not match, even though the hex codes are the same.

comment:44 by skyper, 3 years ago

I see, and think this is rather confusing as the color will not look the same if e.g. imagery has lots of different colors.

comment:45 by skyper, 3 years ago

On little remark about the preset_link. Please add text="Similar but different tags:" as option to <preset_link /> to overwrite the default text. Thanks.

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

in reply to:  45 comment:46 by taylor.smock, 3 years ago

Replying to skyper:

On little remark about the preset_link. Please add text="Similar but different tags:" as option to <preset_link /> to overwrite the default text. Thanks.

I had that initially, but removed it since it didn't make much sense to me (I looked for something like text="Similar preset" -- Similar but different tags seemed like it was something for contact:email <-> email pairings, and power=pole <-> man_made=utility_pole + utility=power, but doesn't make sense for man_made=utility_pole + utility=telecom).

comment:47 by skyper, 3 years ago

Ok, convinced. Thanks for your work.

by taylor.smock, 3 years ago

Attachment: 20270.6.patch added

Drop opacity, add text= to preset_links

in reply to:  47 comment:48 by taylor.smock, 3 years ago

Replying to skyper:

Ok, convinced. Thanks for your work.

No problem -- its something Mapillary has been bugging me about for awhile (they want me to enable smart add capability for utility pole detections, which is blocked by needing a preset to show the user).

by taylor.smock, 3 years ago

Attachment: 20270.7.patch added

Change preset_link text from Similar but different tags to Similar preset. (AKA: New string for translation)

in reply to:  33 ; comment:49 by skyper, 3 years ago

Replying to skyper:

  • in #20530, I used pink (source:trunk/resources/images/presets/landmark/pump_sewage.svg) for substance=sewage which should probably be changed to a darker blue.

See #21388.

in reply to:  49 comment:50 by skyper, 3 years ago

Replying to skyper:

Replying to skyper:

  • in #20530, I used pink (source:trunk/resources/images/presets/landmark/pump_sewage.svg) for substance=sewage which should probably be changed to a darker blue.

See #21388.

In r18240 the background color was changed to #1d5575, so we need to update sewerage.

node|z17-[man_made=utility_pole][utility=sewerage]::utility_casing {
utility_casing: #1d5575;
}

in reply to:  43 ; comment:51 by francois.lacombe, 3 years ago

Hi all

Replying to skyper:

Nice, you used mapcss for the background.

That's nice indeed.
Is it possible to do the same for other icons (not urgent but just to think about for further improvements)?

I wonder if all halos can't be as small as no-utility one?
There may be numerous poles in the same place and this could clutter display too much.
No-utility could remain with white casing and no halo, how do you feel with that?

Note that, regarding utility used on poles, utility=telecom;television is not possible, wiki states it is expected with a single value.

Replying to taylor.smock:

They were synced already. Except for the telecom street_cabinet color.

We plan to replace street_cabinet=* by utility=* in a few months, will be discussed in a proposal
No need to spend so much time on street_cabinet for now.

It's impressive to have such constructive work on a pretty complex topic, that's really encouraging :)

Best regards

in reply to:  51 ; comment:52 by skyper, 3 years ago

Replying to francois.lacombe:

Replying to skyper:

Nice, you used mapcss for the background.

That's nice indeed.
Is it possible to do the same for other icons (not urgent but just to think about for further improvements)?

Generally speaking, sure. Uses less icons and makes it easier to maintain, though the icons have to fit and the form of the halo is restricted. For presets this does not work, yet, and I did not test icons in panels or other dialogs.

I wonder if all halos can't be as small as no-utility one?
There may be numerous poles in the same place and this could clutter display too much.
No-utility could remain with white casing and no halo, how do you feel with that?

Do you know some area to test it?

Note that, regarding utility used on poles, utility=telecom;television is not possible, wiki states it is expected with a single value.

I do not find this written on the page and I wonder how you want to express that there are multiple utilities on the same pole or inside the same service building.

Replying to taylor.smock:

They were synced already. Except for the telecom street_cabinet color.

We plan to replace street_cabinet=* by utility=* in a few months, will be discussed in a proposal
No need to spend so much time on street_cabinet for now.

Nothing changed regarding street cabinets, I just wanted to make sure we use the same color.

It's impressive to have such constructive work on a pretty complex topic, that's really encouraging :)

It depends on both sides. It is much easier to create new presets from well structured wiki pages and proposals, than digging through the wiki history and taginfo to find information about the tags.

comment:53 by Don-vip, 3 years ago

Summary: [PATCH] Add `man_made=utility_pole` to presets[WIP PATCH] Add `man_made=utility_pole` to presets

in reply to:  52 comment:54 by francois.lacombe, 3 years ago

Summary: [WIP PATCH] Add `man_made=utility_pole` to presets[PATCH] Add `man_made=utility_pole` to presets

Replying to skyper:

Generally speaking, sure. Uses less icons and makes it easier to maintain, though the icons have to fit and the form of the halo is restricted. For presets this does not work, yet, and I did not test icons in panels or other dialogs.

Valid indeed, I'll try to remember of that.

Do you know some area to test it?

Possibly in Annecy, France
https://gespot.fr/#15.25/45.908831/6.116115

I do not find this written on the page and I wonder how you want to express that there are multiple utilities on the same pole or inside the same service building.

It regards man_made=utility_pole (and power=pole implicitly) only for now
https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dutility_pole

I think this could be extended to utility=* in further proposal as street cabinets, poles, markers... often are intended for a single utility with possible side features on them.

This is how multi matching is done (or planned to)
https://wiki.openstreetmap.org/wiki/Tag:man_made%3Dutility_pole#Logical_link_between_poles_and_supported_features

comment:55 by skyper, 3 years ago

Summary: [PATCH] Add `man_made=utility_pole` to presets[WIP-PATCH] Add `man_made=utility_pole` to presets

I see, only the primary utility for now. Think there should be an option to add the secondar utilities, too.

by skyper, 3 years ago

Attachment: josm_20270_8.patch added

version 8: mapcss: new color for sewerage; remove duplicate gas; support only single value for telecom/television

comment:56 by skyper, 3 years ago

I have attached version 8:

  • own color for sewerage
  • remove duplicate rendering for gas
  • only support single value

Did not play with the size of casing, yet.

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

in reply to:  51 comment:57 by skyper, 3 years ago

@Taylor:
What do you think about the smaller radius of the halo in general as Francois suggested?

Replying to francois.lacombe:

I wonder if all halos can't be as small as no-utility one?
There may be numerous poles in the same place and this could clutter display too much.
No-utility could remain with white casing and no halo, how do you feel with that?

comment:58 by taylor.smock, 3 years ago

I'm ok with that.

The casing size was originally due to making it transparent, and anything much smaller wasn't easy to see. It has probably changed since transparency was removed for the icon halo.

I do think 10 might be a little to small, but I'm also not mapping hundreds of these. It might be good enough. But I'd probably try 12 instead of 10.

by taylor.smock, 3 years ago

Attachment: 20270.size_12.patch added

Same as attachment:josm_20270_8.patch, but halos are size 12 (static)

comment:59 by skyper, 3 years ago

Summary: [WIP-PATCH] Add `man_made=utility_pole` to presets[PATCH] Add `man_made=utility_pole` to presets

Fine, think we are done and ready for commit.

comment:60 by Don-vip, 3 years ago

Milestone: 21.0921.10

Milestone renamed

comment:61 by Don-vip, 3 years ago

Resolution: fixed
Status: newclosed

In 18270/josm:

fix #20270 - Add man_made=utility_pole (patch by taylor.smock and francois.lacombe, reviewed by skyper)

in reply to:  61 ; comment:62 by skyper, 3 years ago

Replying to Don-vip:

In 18270/josm:

fix #20270 - Add man_made=utility_pole (patch by taylor.smock and francois.lacombe, reviewed by skyper)

Credit belongs to Taylor. Myself, did not change anything.

in reply to:  62 comment:63 by taylor.smock, 3 years ago

Replying to skyper:

Credit belongs to Taylor. Myself, did not change anything.

Images were from myself (attachment:pole.svg, literally the power pole icon with a white border instead of a golden-yellow border) and francois.lacombe (attachment:pole_street_light.svg).

I did change the preset and mapcss files, but skyper gave feedback. And removed a dupe in attachment:josm_20270_8.patch.

Anyway, I'm perfectly OK with you (skyper) getting credit. I kind of wish that francois.lacombe got mentioned for his street light svg, but this is a longish ticket, so I can understand missing stuff like that. I probably should try (in the future) to indicate that in the attachment text, except I usually use that to indicate what changed from the last patch.

comment:64 by Don-vip, 3 years ago

I've updated the commit message.

in reply to:  64 comment:65 by taylor.smock, 3 years ago

Replying to Don-vip:

I've updated the commit message.

Thanks. Hopefully this doesn't break the git mirrors.

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.