acts icon indicating copy to clipboard operation
acts copied to clipboard

refactor: Remove `lastHierarchySurfaceReached` from `Navigator`

Open andiwand opened this issue 1 year ago • 5 comments

This is a leftover from previous PRs which cleaned out the Navigator

andiwand avatar May 30 '24 15:05 andiwand

Codecov Report

Attention: Patch coverage is 50.00000% with 1 line in your changes missing coverage. Please review.

Project coverage is 47.36%. Comparing base (d8f615f) to head (4663a6f). Report is 121 commits behind head on main.

Files Patch % Lines
Core/include/Acts/Propagator/Navigator.hpp 50.00% 1 Missing :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3237      +/-   ##
==========================================
- Coverage   47.59%   47.36%   -0.24%     
==========================================
  Files         507      511       +4     
  Lines       29137    30257    +1120     
  Branches    13976    14679     +703     
==========================================
+ Hits        13869    14332     +463     
- Misses       5269     5388     +119     
- Partials     9999    10537     +538     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov[bot] avatar May 30 '24 18:05 codecov[bot]

I will update the ref since the flag was dealing with a different kind of navigation I suppose the new output is more meaningful

andiwand avatar Jun 13 '24 11:06 andiwand

:red_circle: Athena integration test results [3d85e3c22ff61064c6452ee3e4c75db868fa1b51]

:red_circle: Some tests have failed!

Please investigate the pipeline!

status job report
:green_circle: report_pull_request
:green_circle: run_art_test: test_data18_13TeV_1000evt
:green_circle: run_art_test: test_ttbarPU40_reco
:green_circle: test_ActsDumpGeometryIdentifiers
:red_circle: test_ActsCheckObjectCountsCachedWorkflow
:red_circle: test_ActsCheckObjectCountsWorkflow
:green_circle: test_ActsEFTrackFit
:green_circle: test_ActsPersistifySeeds
:green_circle: test_ActsBenchmarkWithSpot
:red_circle: test_ActsAnalogueClustering
:red_circle: test_ActsWorkflowHeavyIons
:red_circle: test_ActsWorkflowFastTracking
:red_circle: test_ActsWorkflowCached
:red_circle: test_ActsWorkflow
:green_circle: test_ActsValidateAmbiguityResolution
:red_circle: test_ActsValidateResolvedTracks
:red_circle: test_ActsValidateTracks
:green_circle: test_ActsValidateActsCoreSpacePoints
:green_circle: test_ActsValidateActsSpacePoints
:green_circle: test_ActsValidateSeeds
:green_circle: test_ActsValidateOrthogonalSeeds
:green_circle: test_ActsValidateClusters
:red_circle: test_ActsPersistifyEDM
:green_circle: test_ActsGSFRefitting
:green_circle: test_ActsKfRefitting
:green_circle: test_ActsExtrapolationAlgTest
:green_circle: test_ActsITkTest
:green_circle: run_workflow_tests_run4_mc
:green_circle: run_workflow_tests_run2_mc
:green_circle: run_workflow_tests_run2_data
:green_circle: run_workflow_tests_run3_mc
:green_circle: run_workflow_tests_run3_data
:green_circle: run_unit_tests

acts-project-service avatar Jun 13 '24 14:06 acts-project-service

Still failing - this might be something specific about the ITk geometry. I will try to reproduce this in the standalone outside Athena. Very weird

andiwand avatar Jun 14 '24 08:06 andiwand

There are some count changes in Athena. I will change the reference on the canary as soon as this goes in

andiwand avatar Jul 24 '24 11:07 andiwand