-
Notifications
You must be signed in to change notification settings - Fork 822
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
Added rendering for ref on public_transport=stop_positions #4800
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not a full review - just a few technical issues, including those related to problems you were experiencing.
@pnorman Thanks! |
From my side this pull request is ready to be merged. |
Thanks for the pull request. This is not a review but i like to mention that as a general principle we do not want to newly introduce interpretation of several tags as synonyms in this style when mappers have not reached a consensus on tagging. If two tags are consistently used for different things the option to render them identically is there - but i don't see this being the case here. Beyond that i would like to see input from people experienced in public transport mapping on this. What is shown here in the examples is the local numbering/labeling of tracks or platforms within train stations. It would be important to know if the interpreted tagging is commonly used for that in different parts of the world. I also wonder what the conventions are for the location of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There remain SQL style issues and ordering problems raised in previous reviews.
Only one of ref
or local_ref
should be used. My preference is ref
.
The position of this layer relative to others doesn't make much sense. Right now it's lower priority than only place names and stations. This will lead to other features being rendered up to zoom 18, and then train stop positions will come up earlier in the rendering and collide them away. This is sometimes okay, but generally not a good idea because it's confusing to have features disappear like that.
Given the size and zoom, my inclination would be to put these either before or after the *-low-priority layers.
Note: I haven't reviewed the MSS or cartography yet, but the issues raised here are independent of those.
Thanks for the quick review, I fixed the SQL style, changed the PR to only render ref and not local_ref and moved the priority befor the low-priority layers. @imagico I'm not an expert for public transprot mapping, but as far as I now placing the |
@pnorman : "Only one of ref or local_ref should be used. My preference is ref." The 'ref' should be seen as the global, unique ID for the stop (in the country?) and does not reflect the platform/track of the station. The 'local_ref' is the the platform/track number of the station which should be rendered. |
@imagico : "I also wonder what the conventions are for the location of public_transport=stop_position nodes." I agree to the view of @map-per : this is mostly seen as the middle of the train/platform - for trains. stop_positions for buses are different - mostly where the front door of the bus will be when the bus stops. |
Maybe i need to explain my request more clearly. I am asking about mapping practice to avoid the need to spend time analyzing tag use of features i am unfamiliar with and this way to facilitate a faster review. But this only works if i can rely on the answers i get. I had a look myself now - growing a bit suspicious with the conflation of statements about de facto and ought-to-be tag use in the answers and the result is that while placement of https://www.openstreetmap.org/node/10027694711 So i restate my request: I would like to see input from people experienced in public transport mapping on this. That means critical assessments of people knowledgeable in the domain about this change and the assumptions it makes on mapping and tagging practice, not just affirmative statements by people who would like to see this change merged. As said: This change aims to label the local numbering/labeling of tracks or platforms within train stations. Therefore it is important to know if and to what extent the assumptions this makes on mapping and tagging practice for that purpose are actually reflecting consensus among mappers in various parts of the world and to what extent and where mapping conventions exists that differ and therefore would clash with this. It would also be useful to have a review of how other maps with global scope (in particular public transport focused ones) handle the numbering/labeling of tracks or platforms within train stations. |
@imagico: ToniE is experienced in public transport mapping, he even runs his own public transport quality assurance tool (https://ptna.openstreetmap.de). That's why I asked him to take a look at this. @osm-ToniE: Thanks for taking a look at this! |
@imagico Is there anything else I can contribute here? (if so, I would be very happy to receive an e-mail in German). |
What is needed from my perspective here is a thorough analysis of the world wide mapping practice. There are different concepts of numbering (platform/stop/track/route numbers with different scope of application) and there are different methods to record such numbers in OSM. These seem primarily:
In some cases these are further differentiated into simple numbers and semicolon separated lists. In addition - as mentioned above - the placement practice of To decide if we can productively show any of these and how we should possibly do so we would need to know in more detail how these different methods are used to record what kind of numbers and how this practice of use differs globally. I have only looked at this enough to see that there is substantial variability that we cannot ignore. But that does not necessarily mean we can't render anything. But we need reliable information on this to determine viability. Feel free to provide information in languages other than English if that is easier - we will try to make use of that. If reading English is an issue please try using online translation services (like deepl). |
@imagico : Another mapper drew my attention to www.openrailwaymap.org. Does this possibly help with a solution? : https://www.openrailwaymap.org/?lang=de&lat=51.8815775618944&lon=14.063186645507812&zoom=10&style=standard |
@imagico there seems to be no common sense on that. Even inside the station München Hauptbahnhof, shown in the first comment, it is not consistent.
I would use local_ref on public_transport=stop_position as priority 1
If local_ref is not set, one might consider using ref instead?
My 2ct: use local_ref on public_transport=stop_position and propagate/postulate/announce this as being 'the choice' for Carto |
@osm-ToniE , @imagico : Thank you for addressing this issue. It would be really helpful if there was a solution. |
Fixes #4662
Changes proposed in this pull request:
Test rendering with links to the example places:
(before left, right after)
https://www.openstreetmap.org/node/2470201868
https://www.openstreetmap.org/node/2465304880
https://www.openstreetmap.org/node/2476438979
https://www.openstreetmap.org/node/3135325888
https://www.openstreetmap.org/node/260406611
https://www.openstreetmap.org/node/2499357757
https://www.openstreetmap.org/node/3707303125
https://www.openstreetmap.org/node/3183841635