Rendering of Chinese CTCS signalling #115
Conversation
|
Regarding the color: at small zoom levels it is expected that three main areas of homogeneous signalling emerge, Europe USA and China. Maybe India and Russia at a later stage. So, 5 colors max? They should really not be any similar to each other, mainly because of the legend. ETCS is blue and well established, compatible with the other many regional colors. Options for CTCS are green, yellow/orange, or red as asked from local contributors. The use of red, at any shade, should in my opinion go together with a recoloring of PTC, which I think is at this time still possible. A possible combination: I'm asking for a feedback from the maintainers about whether or not changing the PTC color. If not, I would swap PTC and CTCS from the list above. |
|
What about add this to #118 now or later? |
|
It could be added to PR#118. However I need
|
In fact, I have been checking the information these days and deleted many labels that were wrongly labeled as ETCS on the railways in China.
Usually not quickly :sad: |
|
Are you already checking the tags? Then could you add railway:train_protection=CTCS to all the lines you edit? This way if I incorporate the changes in PR#118 I have data for testing. |
What I removed was the I will go immediately check OSMWiki to determine how to tag it. (Sorry, I also ask https://t.me/OpenRailwayMap/19/897 for some suggestion) |
|
According to the proposal, version numbers can be added like this: railway:train_protection=ETCS:1LS The new tagging system looks for "CTCS" or "ETCS" in the string, so version number won't affect the rendering (unless in the future we decide to). |
There seems to be a discrepancy between this and what I've heard within the editor community, perhaps we need to look to the ORM mailing list and tagging for a larger discussion?
As far as the OSM Wiki is referenced, the form of
Because CTCS is not documented in OSM Wiki (but it does not mean it does not exist, there are more than 20,000 ways), so this is the data of taginfo
I admit that But it takes time to go from proposal to promotion (given the cooperative model of the open source community, this usually takes several years), and what we are facing is the current status of OSM data. |
I'm happy to be referred to an active mailing list :-)
Yes.
Unfortunately many things are not documented, other things are documented but in practice do not exist.
I think as long as you add things you have margin of action. Tags in OSM are full of things like A:B:construction and A:construction:B, and sooner or later consensus is reached and a bot is created to uniform the situation. I don't see a problem with the proposal. |
|
IMHO there is no possibility to get even the "big" systems not not share colors somehow. ETCS will likely be somewhat global, and I expect the same happening for CTCS. So if CTCS would be red as the German LZB is I don't expect any clashes, as the CTCS will not likely start to exist in Germany, and LZB is waiting for it's removal (there are no new installations). But for somewhat local systems like any of the other existing European systems I personally would say they only need to be distinct from the color of their neighbor countries unless there is a big company behind them selling them worldwide, in which case we would need to discuss. At the end you will have similar colored systems in the legend. On an unrelated note I once tried to have geofenced legend entries, i.e. if you zoom in into Europe then CTCS would be omitted from the legend or vice versa. I never really finished that, but I would love to see someone picking this up again. But before that we really need a way to get the legend use the CartoCSS style. |
|
I think this introduces a regression where all tracks rendered as previously unknown are now rendered as having no train protection. |
|
Regression seems confirmed... honestly, I don't understand why. And to say everything, it's the fourth time we run into the same issue of having to check all possible combinations to detect the lack of signalling. My proposed fix is in the other PR that uses the proposed new tagging system #118 |
Yes, this is the right way forward imho. |
|
@davide84 don't forget to add "CBTC" to the Legend also. |
This reverts commit c0cc032. The commit introduced a regression documented in #115 (comment)
|
I don't know why I did not notice the black lines in my own test renderings. I reverted the commit, deployed and started rerendering it a few minutes ago. |
It's because of the combination of |
7 participants
Added support for CTCS following discussion here:
#37
TODO: finalize color, we can discuss here.