-
Notifications
You must be signed in to change notification settings - Fork 128
Update public transport tags for STAN french network in analyser #2603
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
base: master
Are you sure you want to change the base?
Conversation
|
Up ? Thank you ! |
|
@nlehuby Can we have your review of this PR, please ? |
nlehuby
left a comment
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.
Hello,
Bus stops here seem to be mapped in a rather unusual way compared to what is done in France (see in particular https://wiki.openstreetmap.org/wiki/FR:Bus#Arr%C3%AAt_de_bus), with a highway=bus_stop on the public_transport=stop_position, and an isolated public_transport=platform, mapped as a polygon.
This does not seem very sound to me (even if it remains broadly compatible with the approved transport scheme). Unless this is the result of a concerted decision by the local community, I would suggest cleaning up this point before carrying out open data integrations.
Regarding integration:
- The ref tag must correspond to a reference or identifier visible in the field. Is this the case here: can we see the actual GTFS stop_id displayed at bus stops?
- Is it really useful to add gtfs:location_type:FR-GES-STAN=0 to all stops?
- Is there a difference between the names displayed in the field and the names in the GTFS that would justify having both name and gtfs:stop_name:FR-GES-STAN tags?
|
Hello, Thank you for your feedback, I followed PTV2 without fully integrating the common attributes, Regarding the GTFS tags:
I will clean up the mapping and the tags based on these points. |
|
I followed the ptv2 and the gtfs pages on the wiki : https://wiki.openstreetmap.org/wiki/GTFS#Step_3:_Reference_the_object
|
|
Regarding the model, what is unusual is not the use of public_transport=platform, but rather
The usual model generally proposes:
And the examples reported by your overpass query correspond to train platforms and not bus stops, so the comparison is a little biased. Regarding the open data integration, thanks for the changes. |
|
|
Please note, I never said that your mapping choice was wrong, I'm just saying that it's not the most common method in France. I'm just saying that your way of mapping stops on this network is unusual. In concrete terms, if a non-local contributor passes by and spots a missing bus stop, whether they are a beginner or experienced in the topic of public transport, I don't think they will naturally map that missing stop with an area with public_transport=platform: this is not the most common practice. So there is a risk of developing inconsistencies within the network over time, which will complicate its maintenance for the contributors and the usability of the data. In any case, you are free to stick with your current approach, because it is indeed compliant with the approved schema, which focuses primarily on the use of the public_transport tag. |
|
Thanks for the advice! So the best approach should be to :
Does this sound right? Thanks again! |
|
Thank you again for your valuable feedback. According to GTFS, we will keep GTFS tags on The commit implementing these changes is ready. Best regards, |




Hello,
We recently started adding the
gtfs:stop_id:FR-GES-STAN=tags to platform elements based on GTFS and PTNA data. However, the current Osmose analyser does not recognize these stops, so adjustments are required for proper detection.Previous tags:

New tags:

Example screenshot:

Would you mind reviewing my PR and confirming if the changes meet the requirements for Osmose integration? Thank you for your time!