Skip to content

Conversation

louisgreiner
Copy link
Contributor

@louisgreiner louisgreiner commented Aug 22, 2025

still work in progress

Can be review until commit: asymmetry: add symmetry TrainrunSection toggle + enable to reset the symmetry on perlenkette-section component

Missing features:

  • Non-stop trainruns
    • Perlenkette times updates for non-stop

Some bugs (shaky features) to fix:

  • reset the whole trainrun symmetry from trainrun tab sometimes lead to wrong time values
  • symmetry selection after symmetry toggle click on perlenkette-section component changes the times in the background before clicking on Confirm
  • BackwardTravelTime text position on reticular

Ref #418

For reviewing: https://sncf.sharepoint.com/:p:/r/sites/OSRD644GrpO365/_layouts/15/Doc.aspx?sourcedoc=%7B68C024EF-2C05-4B4D-86C1-ED205F966F2F%7D&file=Cas%20modifs%20temps%20NGE.pptx&action=edit&mobileredirect=true

@louisgreiner louisgreiner mentioned this pull request Aug 25, 2025
36 tasks
@louisgreiner louisgreiner force-pushed the lgr/asymmetry branch 15 times, most recently from fb51a6f to 6293d17 Compare August 29, 2025 16:54
@emersion emersion self-requested a review September 10, 2025 08:31
@louisgreiner louisgreiner force-pushed the lgr/asymmetry branch 4 times, most recently from d84931e to ae65fdb Compare September 19, 2025 14:40
@aiAdrian aiAdrian added this to the 2.11 milestone Sep 24, 2025
@louisgreiner louisgreiner force-pushed the lgr/asymmetry branch 6 times, most recently from 01324b1 to 43e67a3 Compare October 3, 2025 08:21
…y on trainrun-tab component

Enables to reset the symmetry on the whole trainrun at once.

Signed-off-by: Louis Greiner <[email protected]>
…y on perlenkette component

Signed-off-by: Louis Greiner <[email protected]>
…symmetry on perlenkette-section component

Also includes a refactor (sorry for the diff) because this component was impossible to work with otherwise.

Signed-off-by: Louis Greiner <[email protected]>
Signed-off-by: Louis Greiner <[email protected]>
Signed-off-by: Louis Greiner <[email protected]>
Signed-off-by: Louis Greiner <[email protected]>
Had to nuke a lot a existing code to make non-symmetry fit in it. Created a BackwardIterator for that also. Non-stop fully symmetrical trainruns work. Non-stop asymetrical trainruns are not working yet.

Signed-off-by: Louis Greiner <[email protected]>
Did not test with asymmetric trainruns, only fixed for regular trainruns. Travel time update looks still broken too (but wasn't it already?).

Signed-off-by: Louis Greiner <[email protected]>
…are not coherent due to asymmetry

Removed the source->target legacy check. This component should be refactored once more, it's still hard to read.

Signed-off-by: Louis Greiner <[email protected]>
…times become non symmetrical

Signed-off-by: Louis Greiner <[email protected]>
…tions chain (non-stop case)

Signed-off-by: Louis Greiner <[email protected]>
This can be easily refactored.

Signed-off-by: Louis Greiner <[email protected]>
@louisgreiner louisgreiner force-pushed the lgr/asymmetry branch 4 times, most recently from b489bda to 2c03b21 Compare October 19, 2025 18:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants