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

revert: "refactor: Use common direction transform Jacobian (#2782)" #2855

Closed

Conversation

paulgessinger
Copy link
Member

@paulgessinger paulgessinger commented Dec 25, 2023

This reverts commit 7b26be2 from #2782, which seems to cause a significant amount of new navigation failures. Opening this so I can easily run the Athena tests.

@paulgessinger paulgessinger added this to the next milestone Dec 25, 2023
@github-actions github-actions bot added Component - Core Affects the Core module Component - Examples Affects the Examples module Changes Performance labels Dec 25, 2023
Copy link

codecov bot commented Dec 25, 2023

Codecov Report

Attention: 54 lines in your changes are missing coverage. Please review.

Comparison is base (7b26be2) 48.83% compared to head (f427173) 48.87%.

Files Patch % Lines
Core/src/Surfaces/DiscSurface.cpp 6.89% 15 Missing and 12 partials ⚠️
Core/src/Surfaces/Surface.cpp 14.28% 0 Missing and 12 partials ⚠️
Core/src/Surfaces/LineSurface.cpp 0.00% 0 Missing and 9 partials ⚠️
Core/src/Propagator/detail/CovarianceEngine.cpp 83.33% 5 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2855      +/-   ##
==========================================
+ Coverage   48.83%   48.87%   +0.04%     
==========================================
  Files         486      485       -1     
  Lines       28175    28205      +30     
  Branches    13285    13286       +1     
==========================================
+ Hits        13760    13786      +26     
- Misses       4820     4829       +9     
+ Partials     9595     9590       -5     

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

@acts-project-service
Copy link
Collaborator

🔴 Athena integration test results [f427173]

🔴 Some tests have failed!

Please investigate the pipeline!

status job report
🟢 run_unit_tests
🔴 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsBenchmarkWithSpot.sh 8 100
🔴 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsWorkflow.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateAmbiguityResolution.sh
🔴 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateResolvedTracks.sh
🔴 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateTracks.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateActsCoreSpacePoints.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateActsSpacePoints.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateSeeds.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateOrthogonalSeeds.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsValidateClusters.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsPersistifyEDM.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsGSFRefitting.sh
🟢 run_ci_tests: ../athena/AtlasTest/CITest/test/ActsKfRefitting.sh
🟢 run_ci_tests: python3 ../athena/Tracking/Acts/ActsGeometry/test/ActsExtrapolationAlgTest.py
🟢 run_ci_tests: python3 ../athena/Tracking/Acts/ActsGeometry/test/ActsITkTest.py
🟢 run_workflow_tests_run4_mc
🟢 run_workflow_tests_run2_mc
🟢 run_workflow_tests_run2_data
🟢 run_workflow_tests_run3_mc
🟢 run_workflow_tests_run3_data
🟢 run_art_test: test_data18_13TeV_1000evt 📎
🟢 run_art_test: test_ttbarPU40_reco 📎

@acts-project-service acts-project-service added the Fails Athena tests This PR causes a failure in the Athena tests label Dec 25, 2023
@paulgessinger
Copy link
Member Author

paulgessinger commented Dec 26, 2023

Ok this still shows failures.

@paulgessinger paulgessinger modified the milestones: next, v32.0.0 Jan 19, 2024
@paulgessinger paulgessinger removed the Fails Athena tests This PR causes a failure in the Athena tests label Jan 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes Performance Component - Core Affects the Core module Component - Examples Affects the Examples module
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants