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

Remove Serbian Railways as it's a holding company #9736

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

u-spec-png
Copy link
Contributor

Serbian Railways is just a holding company Serbian Railways Infrastructure manages infrastructure assets.

@UKChris-osm
Copy link
Collaborator

I don't know anything about the railways in Serbia, but the current entry within the NSI has lots of POI's added to the map already, whereas this suggested edit seems to have none so far.

Could this change cause a conflict within Serbian mappers??

@Dimitar5555
Copy link
Collaborator

I don't know anything about the railways in Serbia, but the current entry within the NSI has lots of POI's added to the map already, whereas this suggested edit seems to have none so far.

It's very common to tag the passenger company as operator of the infrastructure since people are not aware that they are operated by different entities. It's also understandable, since for the last 40 or more years there was a single company which managed infrastructure, rail freight and rail passengers.

P.S. @u-spec-png, if you are not going to update all route=railway relations in Serbia after this PR is merged, it would be nice include "matchNames":["Железнице Србије"],.

@Snowysauce Snowysauce added check local This issue should be checked with the local mapping community for guidance needs discussion Waiting for other contributors to voice their opinion labels Sep 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
check local This issue should be checked with the local mapping community for guidance needs discussion Waiting for other contributors to voice their opinion
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants