Skip to content

editoast, front: fix respect margin#16206

Open
Synar wants to merge 2 commits intodevfrom
ali/fix-margins-editoast-new-table-front
Open

editoast, front: fix respect margin#16206
Synar wants to merge 2 commits intodevfrom
ali/fix-margins-editoast-new-table-front

Conversation

@Synar
Copy link
Copy Markdown
Contributor

@Synar Synar commented Apr 11, 2026

Close #16152

@Synar Synar requested review from a team as code owners April 11, 2026 00:50
@github-actions github-actions bot added area:front Work on Standard OSRD Interface modules area:editoast Work on Editoast Service labels Apr 11, 2026
Copy link
Copy Markdown
Member

@flomonster flomonster left a comment

Choose a reason for hiding this comment

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

Thanks for this fix 🙏

@Synar Synar force-pushed the ali/fix-margins-editoast-new-table-front branch from 75be82f to 3beb10f Compare April 13, 2026 15:38
@Synar Synar requested a review from flomonster April 13, 2026 15:42
@Synar
Copy link
Copy Markdown
Contributor Author

Synar commented Apr 13, 2026

Fixed the tests (they were only broken by the second commit), and reworked one of them in order to test our behavior of checking margins only from one path steps with an arrrival time to the next, skipping other path steps (this new test would have caught the bug fixed in the first commit).

@Synar Synar requested a review from Math-R April 13, 2026 15:45
Signed-off-by: Alice Khoudli <alice.khoudli@polytechnique.org>
@Synar Synar force-pushed the ali/fix-margins-editoast-new-table-front branch from 3beb10f to d69dc03 Compare April 13, 2026 21:46
Signed-off-by: Alice Khoudli <alice.khoudli@polytechnique.org>
@Synar Synar force-pushed the ali/fix-margins-editoast-new-table-front branch from d69dc03 to a612a45 Compare April 13, 2026 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:editoast Work on Editoast Service area:front Work on Standard OSRD Interface modules

Projects

None yet

Development

Successfully merging this pull request may close these issues.

OSRD says the margins are not respected but they are

2 participants