front: fix the warning display above the GET

[clarani]

closes #8878

Add a new boolean allTrainsProjected and display the warning "Projecting..." only if this boolean is not truthy.

[codecov-commenter]

⚠️ Please install the to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 0% with 21 lines in your changes missing coverage. Please review.

Project coverage is 37.13%. Comparing base (8d1935a) to head (0bc4624).
Report is 2 commits behind head on dev.

Files with missing lines	Patch %	Lines
.../components/SpaceTimeChart/useLazyProjectTrains.ts	0.00%	8 Missing ⚠️
...ions/operationalStudies/hooks/useLazyLoadTrains.ts	0.00%	5 Missing ⚠️
...ons/operationalStudies/views/SimulationResults.tsx	0.00%	3 Missing ⚠️
...nalStudies/components/Scenario/ScenarioContent.tsx	0.00%	2 Missing ⚠️
...ations/operationalStudies/hooks/useScenarioData.ts	0.00%	2 Missing ⚠️
...ications/stdcm/hooks/useProjectedTrainsForStdcm.ts	0.00%	1 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files

@@             Coverage Diff              @@
##                dev    #8880      +/-   ##
============================================
- Coverage     37.13%   37.13%   -0.01%     
  Complexity     2211     2211              
============================================
  Files          1260     1260              
  Lines        115076   115085       +9     
  Branches       3230     3230              
============================================
- Hits          42738    42735       -3     
- Misses        70405    70417      +12     
  Partials       1933     1933              

Flag	Coverage Δ
core	74.67% <ø> (ø)
editoast	72.56% <ø> (-0.02%)	⬇️
front	15.20% <0.00%> (-0.01%)	⬇️
gateway	2.20% <ø> (ø)
osrdyne	2.60% <ø> (ø)
railjson_generator	87.49% <ø> (ø)
tests	86.46% <ø> (ø)

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[emersion]

Could we instead use a useMemo with a callback checking shouldProjectIds.length === 0?

[anisometropie]

it doesn’t go away after I just created 100 trains

[anisometropie]

weirdly the too figures are not synchronized 40 on the left 30 on the right

[clarani]

@anisometropie thanks for your review 🙏

• thanks for the bug, I fixed it in the 3rd commit
• for your 2nd remark, actually it's normal that the counters are de-synchronized, since the loading of the trains and the projection are done in parallel and projection is done after the loading

[emersion]

Another way to fix this without a desync would be to subtract the number of invalid trains to the total number of simulated trains. But the current approach here looks fine to me as well.

[anisometropie]

thanks!

[emersion]

LGTM regardless

[kmer2016]

LGTM