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

Link the max speed of the consist to the composition code in STDCM #10016

Closed
1 task
axrolld opened this issue Dec 11, 2024 · 3 comments · Fixed by #10039
Closed
1 task

Link the max speed of the consist to the composition code in STDCM #10016

axrolld opened this issue Dec 11, 2024 · 3 comments · Fixed by #10039
Assignees
Labels
area:front Work on Standard OSRD Interface modules module:stdcm Short-Term DCM

Comments

@axrolld
Copy link
Contributor

axrolld commented Dec 11, 2024

Description

Important

Updated after this comment

Leftover work from #9712

Composition codes can affect the max speed of a train. For freight trains, here is the mapping :

Code Max speed (km/h)
HLP 100
MA80 80
MA90 90
MA100 100
ME100 100
ME120 120
ME140 140
ME160 160
MVGV 200

AC

  • The table above has been implemented and changing the composition code update the max speed if the chosen code is supposed to lower it. User's input are still not overwritten.
@maelysLeratRosso
Copy link

maelysLeratRosso commented Dec 19, 2024

This is the mapping I observe on staging

Code vMax Diff ?
HLP 100 x
MA100 100
MA80 80
MA90 90
ME100 100
ME120 120
ME140 140
MV160 - x
MVGV 200 x

Which is not exactly the one from the table. Did it change @axrolld @Wadjetz ??

The differences are :

  • Value for HLP
  • Value for MVGV
  • For ME160 in the issue table, we have MV160 in the list in the app, and the value is None instead of 160

@axrolld
Copy link
Contributor Author

axrolld commented Jan 9, 2025

My bad i forgot to update the issue main body after this comment. I just did the fix. Good catch !

@maelysLeratRosso
Copy link

Now it's correct then :) And for the points 2 & 3 they seem corrected as well. I validate the ticket !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:front Work on Standard OSRD Interface modules module:stdcm Short-Term DCM
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants