Modify

Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#9169 closed enhancement (fixed)

[patch] New power substations specifications

Reported by: francois.lacombe@… Owned by: team
Priority: major Milestone: 14.01
Component: Internal preset Version: tested
Keywords: substations, power, presets, transformer Cc: on-osm@…

Description

Hello,

The power substations refinement proposal had been accepted. Some presets in JOSM need to be edited or enhanced.

First of all, power=sub_station and power=station are fully deprecated. Warnings are needed.
power=substation (full replacement of power=sub_station) is introduced to map sites where power is transformed. It includes large areas where high voltage power stuff can be found or even small cabinets with local transformers inside.
http://wiki.openstreetmap.org/wiki/Proposed_features/Substation_refinement

Additional things like switches, converters, compensators or even special tag for power lines inside substation are now described.
If wished, I can help to find a good way to have presets in JOSM for these features. Let me know.
voltage=* values are the same as fixed previously in #8847

Mappaint style, especially with power need to be updated.

Polderruner and I are available to answer any questions.

Thanks in advance to support theses changes.

Attachments (4)

power_icons_trimmed.tar.gz (6.5 KB ) - added by skyper 11 years ago.
trimmed icons
josm_9169.diff (9.5 KB ) - added by skyper 11 years ago.
patch
josm_9169_v2.diff (9.0 KB ) - added by skyper 11 years ago.
patch version 2
josm_9169_v3.diff (10.0 KB ) - added by simon04 11 years ago.

Download all attachments as: .zip

Change History (45)

comment:1 by skyper, 12 years ago

I try to find some time to look into it.

One thing we often miss are nice icons. Do you have any or know where to find them ?

comment:2 by francois.lacombe, 12 years ago

I've just written a mail on @talk mailing-list to find someone who can draw a set of consistent icons to identify all features the same way once and for all.

I'll let you know if it's possible to do so or not.

comment:4 by francois.lacombe, 12 years ago

I've mailed Nicolas Mollet with a list of icons which can complete its "power" collection.

Then, JOSM, wiki, renders and other tools will use the same.

Wait & see for availability.

comment:5 by Don-vip, 12 years ago

Ticket #9236 has been marked as a duplicate of this ticket.

in reply to:  4 comment:6 by skyper, 11 years ago

Replying to francois.lacombe@…:

I've mailed Nicolas Mollet with a list of icons which can complete its "power" collection.

Sadly they are only available as png and not as svg.

comment:7 by francois.lacombe, 11 years ago

Good point.

I'm going to ask him if he can produce them in svg instead of bitmap.

He just sent me a set of new items and they suit our needs for now.

I'll let you know when this fresh stuff is available.

comment:8 by francois.lacombe, 11 years ago

He can't produce those icons in svg, there are only available as bitmap.

Is SVG mandatory ?

in reply to:  8 comment:9 by skyper, 11 years ago

Replying to francois.lacombe@…:

He can't produce those icons in svg, there are only available as bitmap.

Is SVG mandatory ?

Not that I know, but I try to use SVG if possible and considering usability it would be easier no matter what size you need.

comment:10 by francois.lacombe, 11 years ago

Understood.
Having different styles and redraw icons with different colors on the fly is a great thing here.
If you don't mind we'll go on with this set. Icons are the same style, can be and actually already used on some renders.

Do you need some extra hints to update presets ?

comment:11 by Don-vip, 11 years ago

These icons are really great, PNG is fine :)

in reply to:  10 comment:12 by skyper, 11 years ago

Replying to francois.lacombe@…:

Understood.
Having different styles and redraw icons with different colors on the fly is a great thing here.
If you don't mind we'll go on with this set. Icons are the same style, can be and actually already used on some renders.

Well, as long as we do not find better suited ones.

Do you need some extra hints to update presets ?

Actually, I was gonna ask on tagging@osm about substation=*. We need some pictures to better illustrate the different values.

comment:13 by skyper, 11 years ago

Owner: changed from team to skyper
Status: newassigned

in reply to:  11 comment:14 by skyper, 11 years ago

Replying to Don-vip:

These icons are really great, PNG is fine :)

Well, wich colour and which style then ?

comment:15 by Don-vip, 11 years ago

Style: simple (the last one).
Colour: The one you prefer :) I trust you :)

comment:16 by francois.lacombe, 11 years ago

We'll have to wait a little more time since all icons aren't drawn yet.

I'm working on this point with Nicolas Mollet directly and let you know when all be ready.

The look and feel is definitely up to JOSM team.

comment:17 by skyper, 11 years ago

Grrr the simple style png contain transparent space one the edges. Gonna have to use imagemagick to cut it off.

I had a look at them with gimp and they are really tiny. Upscaling gets quite fast ugly.

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

by skyper, 11 years ago

Attachment: power_icons_trimmed.tar.gz added

trimmed icons

comment:18 by skyper, 11 years ago

Replying to skyper:

Grrr the simple style png contain transparent space one the edges. Gonna have to use imagemagick to cut it off.

Well convert -trim works but now all images have an individual size. Have a look at the attached archive

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

by skyper, 11 years ago

Attachment: josm_9169.diff added

patch

comment:19 by skyper, 11 years ago

I did upload a first patch.

I did update substation plus added transformer and switchgear + details. I played a bit with icons, though I do not know which tags will get their own new icon.

Please, have a look at it.

Thought we already have icons for power=plant and power=generator in the default style, but did not find them.

comment:20 by Don-vip, 11 years ago

You should rename "Sub Station" to substation ;)

comment:21 by skyper, 11 years ago

I found the icons and did change two more. Overall we could use a lot of icons (generator:method, generator:source ...) if we want to add them to the value list.

Find version 2 with small updates.

by skyper, 11 years ago

Attachment: josm_9169_v2.diff added

patch version 2

comment:22 by skyper, 11 years ago

Think we can apply the patch and update the icons later (one by one).

There are some SVG icons available on the web but no complete set. So no proper solution either.

  • Validator needs to be adjusted along
  • Style needs to be enhanced/updated

comment:23 by skyper, 11 years ago

Owner: changed from skyper to team
Status: assignednew
Summary: New power substations specifications[patch] New power substations specifications

comment:24 by francois.lacombe, 11 years ago

Hi skyper, thank you for the presets update. I'm ok with it.

Regarding icons, Nicolas Mollet and I didn't agree on the right price to pay for some additional icons. We (OSM) should look for another collection since he doesn't want to finalize the project (a bit shame cause the icons had a good looking in previews).

I can help on validator if you want :

  • Warn about station, sub_station deprecation
  • Warn about substation/transformer without operator=*

Anything else ?

For styles, I don't use them so much, maybe others would have useful comments.

comment:25 by Don-vip, 11 years ago

It's a shame we don't deserve some free icons as a well known open source project. Hopefully we don't ask money each time we write some code :D

comment:26 by simon04, 11 years ago

Concerning icons: did you check http://thenounproject.com/en-us/search/?q=power?

comment:27 by skyper, 11 years ago

We can always add better icons later. I am in favour of adding these new presets without new icons.

in reply to:  24 comment:28 by skyper, 11 years ago

Replying to francois.lacombe@…:

I can help on validator if you want :

  • Warn about station, sub_station deprecation
  • Warn about substation/transformer without operator=*

Patches are always welcome but probably better suited in an own ticket.

comment:29 by simon04, 11 years ago

Is the patch ready for inclusion (minus the icons)?

One remark: match="keyvalue" is default for <key> (so <key key="line" value="bay" match="keyvalue" /> is conciser written as <key key="line" value="bay" />).

I'll attach attachment:josm_9169_v3.diff with with the rejected patch hunk manually applied and some minor changes.

by simon04, 11 years ago

Attachment: josm_9169_v3.diff added

comment:30 by Don-vip, 11 years ago

In 6657/josm:

fix #9553, see #9169 - display power=substation

in reply to:  29 comment:31 by skyper, 11 years ago

Replying to simon04:

Is the patch ready for inclusion (minus the icons)?

All approved and used tags are included. - Yes.

One remark: match="keyvalue" is default for <key> (so <key key="line" value="bay" match="keyvalue" /> is conciser written as <key key="line" value="bay" />).

I'll attach attachment:josm_9169_v3.diff with the rejected patch hunk manually applied and some minor changes.

Thanks, still not expert with "match".

At least validator needs to accept the new values. I did create #9555 for this.

comment:32 by Don-vip, 11 years ago

In 6659/josm:

fix #9555, see #9169 - take into account power=substation into power lines validator test

comment:33 by simon04, 11 years ago

Resolution: fixed
Status: newclosed

In 6662/josm:

fix #9169 - New power substations presets - patch by skyper

comment:34 by Don-vip, 11 years ago

In 6664/josm:

see #9169 - fix wiki links

comment:35 by aceman, 11 years ago

Please add context to the combo values in the new presets for proper translation.

comment:36 by simon04, 11 years ago

To which ones exactly?

comment:37 by aceman, 11 years ago

<combo key="substation"
<combo key="location"
<combo key="transformer"

comment:38 by simon04, 11 years ago

In 6673/josm:

see #9169 - Add i18n context to power preset strings

comment:39 by Don-vip, 11 years ago

Milestone: 14.01

comment:40 by Don-vip, 11 years ago

In 6688/josm:

see #9169, fix #9579 - power=substation in deprecated.mapcss

comment:41 by bastiK, 11 years ago

ref #9766

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.