Skip to content
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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

map-per
Copy link
Contributor

@map-per map-per commented Apr 4, 2023

Fixes #4662

Changes proposed in this pull request:

  • Added rendering for ref on public_transport=stop_positions. The track numbers are only shown when train=yes is set
  • Track numbers display from zoom 18 on

Test rendering with links to the example places:
(before left, right after)

https://www.openstreetmap.org/node/2470201868
MHB17
MHB18
MHB19

https://www.openstreetmap.org/node/2465304880
OST17
OST18
OST19

https://www.openstreetmap.org/node/2476438979
Pasing18
Pasing19
Pasing20

https://www.openstreetmap.org/node/3135325888
Neu17
Neu18
Neu19

https://www.openstreetmap.org/node/260406611
Freising18

https://www.openstreetmap.org/node/2499357757
MUC19

https://www.openstreetmap.org/node/3707303125
Fisch18

https://www.openstreetmap.org/node/3183841635
Altona18

Copy link
Collaborator

@pnorman pnorman left a 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.

project.mml Outdated Show resolved Hide resolved
project.mml Outdated Show resolved Hide resolved
project.mml Outdated Show resolved Hide resolved
@map-per
Copy link
Contributor Author

map-per commented Apr 5, 2023

@pnorman Thanks!

@map-per
Copy link
Contributor Author

map-per commented Apr 17, 2023

From my side this pull request is ready to be merged.

@map-per map-per requested a review from pnorman April 17, 2023 15:43
@imagico
Copy link
Collaborator

imagico commented Apr 17, 2023

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 public_transport=stop_position nodes. The way this change renders them incentivizes placing them where it is convenient to have the label, usually probably around the middle of the train when it stops. If mapping conventions suggest a different location (like the front of the train or some other reference point) the mapping incentives we create might clash with that.

Copy link
Collaborator

@pnorman pnorman left a 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.

project.mml Outdated Show resolved Hide resolved
@map-per map-per changed the title Added rendering for ref and local_ref on public_transport=stop_positions Added rendering for ref on public_transport=stop_positions Apr 18, 2023
@map-per
Copy link
Contributor Author

map-per commented Apr 18, 2023

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 stop_position in the middle of the train / paltform is conventional. This is also what I overved when taking a look at stop_positions around the world with overpass.

@map-per map-per requested a review from pnorman April 18, 2023 08:16
@osm-ToniE
Copy link

@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.
This is a more generic name for the stop ID. Other, similar tags are 'ref:IFOPT' and 'uic_ref'

The 'local_ref' is the the platform/track number of the station which should be rendered.

@osm-ToniE
Copy link

@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.

@imagico
Copy link
Collaborator

imagico commented Apr 18, 2023

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 public_transport=stop_position roughly in the middle of the train where it stops is indeed widespread in many countries it is also clear that this is not broad consensus world wide and there are quite a few countries where this is distinctly not the case. A few examples

https://www.openstreetmap.org/node/10027694711
https://www.openstreetmap.org/node/10602804886
https://www.openstreetmap.org/node/4269245658
https://www.openstreetmap.org/node/5836393505
https://www.openstreetmap.org/node/6010790033
https://www.openstreetmap.org/node/1271716495
https://www.openstreetmap.org/node/2367900477
https://www.openstreetmap.org/node/619293646
https://www.openstreetmap.org/node/7070782103
https://www.openstreetmap.org/node/1148313241

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.

@map-per
Copy link
Contributor Author

map-per commented Apr 18, 2023

@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!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Label ref or local_ref on public_transport=stop_position for trains
4 participants