Skip to content
This repository has been archived by the owner on Jul 9, 2023. It is now read-only.

Route Chomutov - Karlovy Vary - Cheb - Schirnding #659

Merged
merged 22 commits into from
Aug 26, 2022

Conversation

embix
Copy link
Contributor

@embix embix commented Aug 22, 2022

  • stations Chomutov - Karlovy Vary
  • route Chomutov - Karlovy Vary
  • stations Karlovy Vary - Cheb
  • route Karlovy Vary - Cheb
  • stations Cheb - Schirnding
  • route Cheb - Schirnding
  • jobs
    • ČD Os 7066 Chomutov - Karlovy Vary
    • ČD Os 7014 Karlovy Vary - Cheb
    • ČD Sp 5282 Cheb - Schirnding - Marktredwitz-Hersbruck
    • OPB RB95 (Oberpfalzbahn) Marktredwitz-...-Cheb

follow up jobs, once required routes are available (deferred to #684 )

Note to self "Regional fast (Sp) and regional (Os) trains will take you to the smallest towns and villages." - Fast (R) trains will transport you quickly from city to city and between regions.

  • ČD Os ist Regional, stoppt überall, nur 2. Klasse - quasi früher P oder aktuell RB (in TC stopsEverywhere=true)
  • ČD Sp ist Regional, stopp nur an wichtigeren Bahnhöfen und Haltepunkten nur 2 .Klasse - früher E bis evtl. D, heute RB bis RE
  • ČD R Fast train 1./2.Kl. - früher D, IR heute RE, REx, IRE
  • ČD Ex Express 1./2.Kl. - früher Ex/SEx

@embix embix changed the title add stations Chomutov - Karlovy Vary - Cheb Route Chomutov - Karlovy Vary - Cheb Aug 22, 2022
embix added 6 commits August 22, 2022 22:05
- move stations relative to previous changes for Chomutov - Karlovy Vary
- retry coords for Chodov from lat/long
- fix station codes
- add platorms/-length
- adjust speed/curves
- also: fix Trsnice station code
@embix
Copy link
Contributor Author

embix commented Aug 23, 2022

@C1710 import_brouter.py mal mit -v ausprobiert (aktuelles dev image, image id 40d5eb142f85)

  • bei Chodov scheint er komische Koordinaten gezogen zu haben
  • bei Hlavno, Dasnice, Kynšperk nad Ohří, Nebanice und Tršnice stimmt der ermittelte CZ-Station Code nicht mit dem aus https://www.cd.cz/stanice überein (bei Hlavno, Dasnice, Kynšperk nad Ohří und Nebanice hatte er das auch als INFO geloggt - bei Tršnice allerdings nicht)
  • Constraint: Kurvigkeit darf 0.5 nicht überschreiten, weniger als 0.1 ist eigentlich schnurgerade Strecke (man könnte mit min-Radien und/oder Häufigkeit minimaler Radien arbeiten - bisher mache ich das nach Bauchgefühl - so ein grober Anhalt/Maßstab/Richtwert für Radius-Kurvigkeit wäre nicht verkehrt)
  • Anzahl und Länge der Bahnsteige ist noch Handarbeit 😄 (ja, ich weiß daß es inzwischen defaults gibt)

@marhei
Copy link
Owner

marhei commented Aug 23, 2022

Meinst du, dass du auch noch die Verbindung zwischen Schirnding und Cheb in den PR mit reinbekommst? 👀

@embix
Copy link
Contributor Author

embix commented Aug 23, 2022

Meinst du, dass du auch noch die Verbindung zwischen Schirnding und Cheb in den PR mit reinbekommst?

Ja, dann wird das aber vermutlich nicht mehr heute fertig.

@embix embix changed the title Route Chomutov - Karlovy Vary - Cheb Route Chomutov - Karlovy Vary - Cheb - Schirnding Aug 23, 2022
@C1710
Copy link
Collaborator

C1710 commented Aug 23, 2022

@C1710 import_brouter.py mal mit -v ausprobiert (aktuelles dev image, image id 40d5eb142f85)

  • bei Chodov scheint er komische Koordinaten gezogen zu haben
  • bei Hlavno, Dasnice, Kynšperk nad Ohří, Nebanice und Tršnice stimmt der ermittelte CZ-Station Code nicht mit dem aus https://www.cd.cz/stanice überein (bei Hlavno, Dasnice, Kynšperk nad Ohří und Nebanice hatte er das auch als INFO geloggt - bei Tršnice allerdings nicht)
  • Constraint: Kurvigkeit darf 0.5 nicht überschreiten, weniger als 0.1 ist eigentlich schnurgerade Strecke (man könnte mit min-Radien und/oder Häufigkeit minimaler Radien arbeiten - bisher mache ich das nach Bauchgefühl - so ein grober Anhalt/Maßstab/Richtwert für Radius-Kurvigkeit wäre nicht verkehrt)
  • Anzahl und Länge der Bahnsteige ist noch Handarbeit 😄 (ja, ich weiß daß es inzwischen defaults gibt)
  1. Kann sein, dass es da wieder Stationen verwechselt hat. Ich ändere es aber vielleicht mal, dass es die Standorte mit denen aus Brouter überschreibt.
  2. Wenn es da was geloggt hat, nimmt es den Code von OpenStreetmaps, weil in den Datensätzen nichts entsprechendes gefunden wurde. Bei Tršnice kann es vielleicht sein, dass es ein etwas anderer Bahnhof ist
  3. Das läuft basierend auf dem Verhältnis Streckenlänge / Luftlinie. Danach ist 0.1 halt ziemlich gerade und erst 0.0 (bzw. 0.01) schnurgerade. Wenn da richtig Schlangen drin sind o.Ä., dann kann es auch schon mal über 0.4 kommen. Ich ändere das aber vielleicht demnächst, dass es eine vereinfachte Form als Grundlage nimmt.
  4. Bahnsteiginfos gibt es nur für DE, FR, selten auch UK und ganz selten aus OSM auch woanders

@embix
Copy link
Contributor Author

embix commented Aug 24, 2022

Wegen den Complaints neulich auf Twitter hab ich mal bei meinem PR nachgeguckt und ein paar Cases gefunden, wo nicht viel mehr als 1km Abweichung drin ist. Stationsabstand ist hier auch gering, so daß es vielleicht 20% wären - geschenkt.

@C1710 Kann es sein, daß du für die Routen bei den Entfernungen Luftlinie nimmst?
Bei größeren Stationsabständen und Gebirge könnte das dann einen größeren Unterschied machen.

@C1710
Copy link
Collaborator

C1710 commented Aug 24, 2022

Wegen den Complaints neulich auf Twitter hab ich mal bei meinem PR nachgeguckt und ein paar Cases gefunden, wo nicht viel mehr als 1km Abweichung drin ist. Stationsabstand ist hier auch gering, so daß es vielleicht 20% wären - geschenkt.

@C1710 Kann es sein, daß du für die Routen bei den Entfernungen Luftlinie nimmst?
Bei größeren Stationsabständen und Gebirge könnte das aber einen großen Unterschied machen.

Also es wird die Summe der Luftlinien zwischen den einzelnen Punkten der Route/des GPX Track-Segments verwendet. Zumindest immer, als ich das nachgesehen habe, hat es gepasst zu den Angaben auf brouter.de...

Die Höhe der Punkte wird nicht berücksichtigt.

@C1710
Copy link
Collaborator

C1710 commented Aug 24, 2022

Wegen den Complaints neulich auf Twitter hab ich mal bei meinem PR nachgeguckt und ein paar Cases gefunden, wo nicht viel mehr als 1km Abweichung drin ist. Stationsabstand ist hier auch gering, so daß es vielleicht 20% wären - geschenkt.

@C1710 Kann es sein, daß du für die Routen bei den Entfernungen Luftlinie nimmst?
Bei größeren Stationsabständen und Gebirge könnte das aber einen großen Unterschied machen.

Also es wird die Summe der Luftlinien zwischen den einzelnen Punkten der Route verwendet. Zumindest immer, als ich das nachgesehen habe, hat es immer gepasst zu den Angaben auf brouter.de...

Sobald es einen twistingFactor von > 0.01 ausgibt, muss die Strecke länger sein als die Luftlinie zwischen den zwei Halten. Also eigentlich kann das kein Problem sein.

@embix
Copy link
Contributor Author

embix commented Aug 24, 2022

Die einzelnen Punkte der Route sind bei mir ja die Zwischenstops/Stationen - ich kann jetzt nicht mehr in die gpx Files gucken (schon gelöscht), ob er auch zwischen meinen Wegpunkten noch Hilfspunkte mitnimmt. In der Oberfläche hat er aber zumindest den Pfad 1:1 zur Bahnstrecke mitgeschwungen.

Ggf. mach ich am WE mal ein Testcase mit einer besonders geschwungenen Strecke - Brockenbahn oder sowas.

Aber zumindest kann ich aktuell nicht bestätigen, daß er deutlich daneben liegt - +/-1 (0.5) km ist bei int als Zielformat im json sowieso Rundungstoleranz.

@C1710
Copy link
Collaborator

C1710 commented Aug 24, 2022

Die einzelnen Punkte der Route sind bei mir ja die Zwischenstops/Stationen - ich kann jetzt nicht mehr in die gpx Files gucken (schon gelöscht), ob er auch zwischen meinen Wegpunkten noch Hilfspunkte mitnimmt. In der Oberfläche hat er aber zumindest den Pfad 1:1 zur Bahnstrecke mitgeschwungen.

Ggf. mach ich am WE mal ein Testcase mit einer besonders geschwungenen Strecke - Brockenbahn oder sowas.

Aber zumindest kann ich aktuell nicht bestätigen, daß er deutlich daneben liegt - +/-1 (0.5) km ist bei int als Zielformat im json sowieso Rundungstoleranz.

Wenn es passt, dann sollte es ja korrekt sein :D

Vielleicht war eine Version bei ner älteren PR auch einfach ein bisschen kaputt oder so?

Also wenn du das exportierst, hast du i.d.R. ein paar Tausend Trackpoints (auf denen halt auch die angezeigte Route basiert).

@embix
Copy link
Contributor Author

embix commented Aug 24, 2022

Also wenn du das exportierst, hast du i.d.R. ein paar Tausend Trackpoints (auf denen halt auch die angezeigte Route basiert).

Ja, dann sollte das passen.

@embix embix mentioned this pull request Aug 25, 2022
42 tasks
@embix embix marked this pull request as ready for review August 25, 2022 19:57
@marhei marhei merged commit f4a6b83 into marhei:main Aug 26, 2022
@embix embix deleted the route-chomutov-cheb branch August 26, 2022 09:02
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants