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.

Sign up for GitHub

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

Jump to bottom

Double computation when setting margin + specific times #7967

Closed
axrolld opened this issue Jul 5, 2024 · 0 comments · Fixed by #7992
Closed

Double computation when setting margin + specific times #7967

axrolld opened this issue Jul 5, 2024 · 0 comments · Fixed by #7992
Assignees
@Akctarus @SarahBellaha
Labels
area:front Work on Standard OSRD Interface modules kind:bug Something isn't working module:operational-studies Multi-train simulation with structured studies management severity:major Major severity bug

Comments

@axrolld
Copy link
Contributor

axrolld commented Jul 5, 2024 •
edited
Loading

What happened?

Asking for some margins + specific times:

image

I got this output:

image

The computed time is correct but it seems like the margins aren't. And the Space-Speed Chart is weird. Both problems seems related.

What did you expect to happen?

The requested time should prevail on the the requested margin. So the real margins should be increased.

And the chart should only have 2 curves anyways (one fastest, one real)

How can we reproduce it (as minimally and precisely as possible)?

  1. Reproduce the screenshots above

What operating system, browser and environment are you using?

  • Browser: Chrome
  • OS: W11
  • Env: dev

OSRD version (top right corner Account button > Informations)

839f7ef
@axrolld axrolld added kind:bug Something isn't working severity:major Major severity bug area:front Work on Standard OSRD Interface modules module:operational-studies Multi-train simulation with structured studies management labels Jul 5, 2024
@Akctarus Akctarus self-assigned this Jul 8, 2024
@SarahBellaha SarahBellaha mentioned this issue Jul 9, 2024
Merged
@SarahBellaha SarahBellaha self-assigned this Jul 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Akctarus Akctarus

@SarahBellaha SarahBellaha

Labels
area:front Work on Standard OSRD Interface modules kind:bug Something isn't working module:operational-studies Multi-train simulation with structured studies management severity:major Major severity bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

front: hide provisional curve in the speed space chart OpenRailAssociation/osrd
3 participants
@Akctarus @axrolld @SarahBellaha