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

front: reset GEV graph each time the selected train changes #3578

Merged
merged 1 commit into from
Mar 20, 2023

Conversation

clarani
Copy link
Contributor

@clarani clarani commented Mar 17, 2023

Closes #3154

  • add an useEffect to reset the GEV graph each time the selected train changes
  • simplify the computation of the scale
  • remove mustRedraw of the GEV (manage the re-drawing of the GEV more locally)

@clarani clarani requested a review from a team as a code owner March 17, 2023 11:13
@clarani clarani marked this pull request as draft March 17, 2023 11:18
@codecov
Copy link

codecov bot commented Mar 17, 2023

Codecov Report

Merging #3578 (1e8ea9d) into dev (3c2cea6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev    #3578   +/-   ##
=========================================
  Coverage     68.36%   68.36%           
  Complexity     1897     1897           
=========================================
  Files           420      420           
  Lines         21206    21206           
  Branches       1535     1535           
=========================================
  Hits          14498    14498           
  Misses         5992     5992           
  Partials        716      716           
Flag Coverage Δ
front 56.89% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@clarani clarani marked this pull request as ready for review March 17, 2023 15:13
@clarani clarani changed the title Cni/front/fix bug 3154 gev front: reset GEV graph each time the selected train changes Mar 17, 2023
@alexandredamiron alexandredamiron merged commit 6e32e92 into dev Mar 20, 2023
@alexandredamiron alexandredamiron deleted the cni/front/fix-bug-3154-gev branch March 20, 2023 14:33
younesschrifi pushed a commit that referenced this pull request Mar 23, 2023
Closes #3154 

- add an useEffect to reset the GEV graph each time the selected train
changes
- simplify the computation of the scale
- remove mustRedraw of the GEV (manage the re-drawing of the GEV more
locally)
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.

front: when the speed chart changes which train is displayed, the distance scale can be off
2 participants