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

Reorder sources.json and line-colors.csv #84

Closed
wants to merge 2 commits into from

Conversation

oneiricbotcelot
Copy link
Contributor

The sorting in line-colors.csv is now strictly alphanumeric for shortOperatorName and lineName. The sorting in sources.json is now strictly alphanumeric for shortOperatorName.

Additonally I did the following very small changes:

  • I removed the prefix STR in the lineName for the trams in Ulm and Freiburg to unify the data corpus for trams. While I initially submitted the colors of both trams back in October, I purposely added STR based on the way Träwelling displayed these lines before line colors were introduced.
  • I changed four shortOperatorNames for lines I previously submitted myself:
    db-regionetz-westfrankenbahn --> db-wfb
    regio-s-bahn-donau-iller --> rsb-di
    sweg-stuttgart --> sweg
    sweg-sudwest --> sweg

...and shortened three shortOperatorNames I previously submitted myself
Sort list strictly alphanumeric (shortOperatorCode and lineName).

Additonally shortened three shortOperatorNames.
@@ -53,24 +53,24 @@ db-regio-nordost,RE66,db-regio-ag-nordost,re-66,#007734,#ffffff,,rectangle
db-regio-nordost,RE7,db-regio-ag-nordost,re-7,#007734,#ffffff,,rectangle
db-regio-sudost,RE14,db-regio-ag-sudost,re-14,#a98956,#ffffff,,rectangle
db-regio-sudost,S1,db-regio-ag-sudost,4-800486-1,#f5de2b,#000000,,pill
db-regio-sudost,S10,db-regio-ag-sudost,4-800486-10,#ffffff,#000000,#f5de2b,pill
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

10 after 1? Yes, I know it's technically correct, but it feels wrong.

For some regions and lines it's fine because line S47 is a shorter or slightly modified version of line S4, but line S10 is probably just the next line after S9 and not a variant of line S1. I'm really not sure if this is the best way to order line numbers...

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I agree, the numbers should be sorted numerically.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The sorting in its current state is absolutely inconsistent and varies between different parts of the dataset. In my opinion this should be fixed. As the technically correct way of sorting doen't seem to be preferred here, I'd like to know how to proceed.

@marhei marhei added the needs-discussion This issue / pull request needs to be discussed. label Jan 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-discussion This issue / pull request needs to be discussed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants