#8434 closed defect (fixed)
[patch] Falsches Preset: man_made=measurement_station - sollte sein: monitoring_station
Reported by: | cmuelle8 | Owned by: | team |
---|---|---|---|
Priority: | normal | Milestone: | 13.12 |
Component: | Internal preset | Version: | latest |
Keywords: | Cc: |
Description (last modified by )
http://wiki.openstreetmap.org/wiki/Tag:man_made=monitoring_station
Vgl. auch
- http://taginfo.openstreetmap.org/keys/man_made=monitoring_station (2066 nodes as of 2013-07-07)
- http://taginfo.openstreetmap.org/keys/man_made=measurement_station (1179 nodes as of 2013-07-07)
Attachments (5)
Change History (42)
comment:1 by , 12 years ago
comment:3 by , 12 years ago
Priority: | major → normal |
---|
follow-up: 5 comment:4 by , 12 years ago
There is a nice part about the truth of numbers on the talk page of measurement_station
.
follow-up: 8 comment:5 by , 12 years ago
Replying to skyper:
There is a nice part about the truth of numbers on the talk page of
measurement_station
.
I don't get your point. You want to use this argument to underline .. what? If you read the talk page to the bottom, it clearly tends to prefer monitoring_station and further suspects: "One of several reasons could be that JOSM has a template for measurement_station." This is why I've opened this bug, josm is at fault here .. since long.
Arguably, osm wiki authors view is subjective and it's not always easy to find a consent in wiki pages, esp. when it comes to talk:-prefixed ones. In this case however, the opinions in the wiki cohere to the (live and wiki-snapshotted) numbers delivered by taginfo. Considering the suspicion vowed above, we should even apply a negative bias to measurement_station numbers before comparing both.
Question remains - why are we sticking to measurement_station here? Esp. when the sole reason of having a significant occurence of this value at all in OSM seems to be the fact, that josm distributed this poor preset in the first place .. and still does?
Thanks for your time.
comment:6 by , 12 years ago
Description: | modified (diff) |
---|
@Don-vip: Danke für die Link-Hinweise - Änderung in die Beschreibung übernommen..
follow-ups: 9 10 comment:8 by , 12 years ago
Replying to cmuelle8:
Replying to skyper:
There is a nice part about the truth of numbers on the talk page of
measurement_station
.
I don't get your point. You want to use this argument to underline .. what? If you read the talk page to the bottom, it clearly tends to prefer monitoring_station and further suspects: "One of several reasons could be that JOSM has a template for measurement_station." This is why I've opened this bug, josm is at fault here .. since long.
Sorry, I did not want to make any judgement or argument but only remind myself and others that numbers do not always tell the truth.
You are right that JOSM should not prefer one of the two tags and if so we should consider using monitoring. I read both pages + talk pages yesterday and did not get the impression that the issue is settled.
I am already involved in some discussions at talk-de@ and tagging@ that's why I did not start another one yesterday but we should get this settled for sure and clean up !
comment:9 by , 12 years ago
Ok, thanks for still watching this.
Truth is man_made, I'm not sure about numbers. ;-)
Greetings
follow-up: 11 comment:10 by , 12 years ago
What about changing this now - considering that the proposal for measurement_station is three years old and nothing substantially changed? Currently, measurement_station does not even have a tag description, while monitoring_station does.
Just looking at the terms I'd prefer measurement myself, but it's not what people use. Maybe we should add a subkey:
monitoring=remote
to indicate that monitoring is not done at the same place where measurement takes place.
Or someone starts dissecting the terms in the wiki and clearly states sth like:
"Monitoring station is a place where humans actually monitor measured data." and
"Measurement station is a place where data is taken, possibly recorded and either collected manually at a certain interval or transmitted by wifi or wire."
In case measurement and monitoring take place on the same site, then, depending on the size of the site, mappers can either decide to micromap the site and still tag measurement and monitoring separately
OR tag it as one of both and specify that the other is done as well by using a dedicated subkey.
Greetings
comment:11 by , 12 years ago
Replying to cmuelle8:
What about changing this now - considering that the proposal for measurement_station is three years old and nothing substantially changed? Currently, measurement_station does not even have a tag description, while monitoring_station does.
As already said, I'd like to first have a conclusion and then clean up and changes. Two or three more weeks won't hurt as we need a transition anyway.
Please open a discussion at tagging@ with your arguments:
Just looking at the terms I'd prefer measurement myself, but it's not what people use. Maybe we should add a subkey:
monitoring=remote
to indicate that monitoring is not done at the same place where measurement takes place.
Or someone starts dissecting the terms in the wiki and clearly states sth like:
"Monitoring station is a place where humans actually monitor measured data." and
"Measurement station is a place where data is taken, possibly recorded and either collected manually at a certain interval or transmitted by wifi or wire."
In case measurement and monitoring take place on the same site, then, depending on the size of the site, mappers can either decide to micromap the site and still tag measurement and monitoring separately
OR tag it as one of both and specify that the other is done as well by using a dedicated subkey.
Thanks.
comment:12 by , 11 years ago
There's a vote in progress:
http://wiki.openstreetmap.org/wiki/Proposed_features/monitoring_station#Voting
To promote monitoring and deprecate measurement. I voted yes.
comment:13 by , 11 years ago
Didn't even know about this ticket until it was mentioned in the voting process for the proposal of monitoring_station.
Thanks for pointing here.
follow-up: 15 comment:14 by , 11 years ago
Vote is finished and everyone approved the proposal => we should then switch to monitoring station and deprecate measurement station :)
comment:15 by , 11 years ago
Replying to Don-vip:
Vote is finished and everyone approved the proposal => we should then switch to monitoring station and deprecate measurement station :)
+1
Please don't forget to update the secondary tagging, too.
comment:17 by , 11 years ago
Resolution: | fixed |
---|---|
Status: | closed → reopened |
Thanks fo the quick work, but the secondary tagging in the preset doesn't match the approved tagging, e.g. recording=manually instead of recording:manually=yes.
comment:18 by , 11 years ago
I didn't notice this part of the proposal :( I thought the only goal was to harmonize measurement and monitoring. What's the point to change display
and recording
tagging ? They don't look broken to me.
comment:19 by , 11 years ago
In short: an example: http://www.openstreetmap.org/browse/way/125685956
More detailed:
The same question was asked and the answer you can find on my blog in the comments (recently added a PS): http://www.openstreetmap.org/user/malenki/diary/20339
comment:21 by , 11 years ago
Hope this gets pulled: https://github.com/openstreetmap/josm/pull/7
(My first try)
comment:22 by , 11 years ago
Pull requests won't be merged since JOSM uses SVN, but will be committed to SVN manually. :-)
comment:24 by , 11 years ago
Please provide SVN patches here, we don't use Github this is just an unofficial mirror.
See DevelopersGuide/PatchGuide for details :)
comment:25 by , 11 years ago
Please, pay attention to white spaces.
We use four for indent and a single one between arguments. No tabs. - Thanks
by , 11 years ago
Attachment: | patch_monitoring_station.diff added |
---|
by , 11 years ago
Attachment: | patch_monitoring_station.patch added |
---|
by , 11 years ago
Attachment: | monitoring_station.patch added |
---|
follow-up: 27 comment:26 by , 11 years ago
*grmbl*
The attachment page was loading so slow I cancelled the upload two times to give the patch a better name... *sigh*
Except for the name the attachments are identical. Take what you like best. :S
follow-up: 28 comment:27 by , 11 years ago
Replying to malenki:
*grmbl*
The attachment page was loading so slow I cancelled the upload two times to give the patch a better name... *sigh*
Except for the name the attachments are identical. Take what you like best. :S
Please, no empty lines.
Why don't we add two checkgroups with three columns, one for recording and one for display ?
by , 11 years ago
Attachment: | monitoring_station.2.patch added |
---|
follow-up: 29 comment:28 by , 11 years ago
Replying to skyper:
Why don't we add two checkgroups with three columns, one for recording and one for display ?
What would the third column be?
Besides, I enhanced the preset as good as I could since no one seemed to have interest to do so. If you want to enhance it further: please do so.
comment:29 by , 11 years ago
Replying to malenki:
Replying to skyper:
Why don't we add two checkgroups with three columns, one for recording and one for display ?
What would the third column be?
For display it would be empty but you are right only two for display are better suited.
Besides, I enhanced the preset as good as I could since no one seemed to have interest to do so. If you want to enhance it further: please do so.
Hey, sorry, I was only asking.
That noone is working on some tickets might have a lot of reasons. One of the first is lack of time. One of the latest is probably a lack of interest.
comment:32 by , 11 years ago
comment:33 by , 11 years ago
Summary: | Falsches Preset: man_made=measurement_station - sollte sein: monitoring_station → [patch] Falsches Preset: man_made=measurement_station - sollte sein: monitoring_station |
---|
comment:34 by , 11 years ago
To me it doesn't matter which patch gets applied as long as tag/value remain correct.
comment:36 by , 11 years ago
Milestone: | → 13.12 (6502) |
---|
Bitte auch den subkey anpassen..
monitoring= statt measurement=
river_level statt water_level
etc.