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: GEV: display power restriction on GEV #3876

Merged
merged 3 commits into from
Apr 18, 2023

Conversation

Yohh
Copy link
Contributor

@Yohh Yohh commented Apr 12, 2023

close #3805

@Yohh Yohh requested a review from a team as a code owner April 12, 2023 13:03
@Yohh Yohh force-pushed the yoh/createPowerRestrictionDisplayOnGEV branch from fc58c8a to cc39c8f Compare April 12, 2023 13:05
@codecov
Copy link

codecov bot commented Apr 12, 2023

Codecov Report

Merging #3876 (c693dbb) into dev (11e692b) will decrease coverage by 0.12%.
The diff coverage is 31.50%.

❗ Current head c693dbb differs from pull request most recent head aced2a5. Consider uploading reports for the commit aced2a5 to get more accurate results

@@             Coverage Diff              @@
##                dev    #3876      +/-   ##
============================================
- Coverage     69.68%   69.56%   -0.12%     
  Complexity     2026     2026              
============================================
  Files           439      439              
  Lines         22245    22309      +64     
  Branches       1672     1672              
============================================
+ Hits          15501    15520      +19     
- Misses         5939     5984      +45     
  Partials        805      805              
Flag Coverage Δ
front 59.07% <31.50%> (-0.22%) ⬇️

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

Impacted Files Coverage Δ
...ront/src/applications/operationalStudies/consts.ts 64.03% <31.50%> (-6.47%) ⬇️

... and 3 files with indirect coverage changes

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

@Yohh Yohh force-pushed the yoh/createPowerRestrictionDisplayOnGEV branch 2 times, most recently from b0c5634 to e9e197b Compare April 12, 2023 15:59
@Yohh Yohh force-pushed the yoh/createPowerRestrictionDisplayOnGEV branch 12 times, most recently from d06b3b5 to 89093fd Compare April 18, 2023 10:11
Yohh and others added 3 commits April 18, 2023 14:18
…etting in SpeedSpaceSettings; update simulation.json

- add condition if used and/or seen restriction is undefined

- fix comments
… from SpeedSpaceChart.jsx; add condition in d3helpers.jsx; edit consts.ts with power restrictions data

- group rects by restriction; add striped pattern

- edit text when incompatible restriction
@Yohh Yohh force-pushed the yoh/createPowerRestrictionDisplayOnGEV branch from 89093fd to aced2a5 Compare April 18, 2023 12:18
@Yohh Yohh merged commit 246312e into dev Apr 18, 2023
@Yohh Yohh deleted the yoh/createPowerRestrictionDisplayOnGEV branch April 18, 2023 12:32
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: Display the power restriction applied on a path
4 participants