docs: Improve documentation on surface classes

[paulgessinger]

In particular LineSurface after #2287.

[codecov]

Codecov Report

Merging #2323 (36ef813) into main (f96b048) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2323   +/-   ##
=======================================
  Coverage   49.57%   49.57%           
=======================================
  Files         453      453           
  Lines       25511    25511           
  Branches    11706    11706           
=======================================
  Hits        12648    12648           
  Misses       4579     4579           
  Partials     8284     8284           

Files Changed	Coverage Δ
Core/include/Acts/Definitions/Direction.hpp	90.24% <ø> (ø)
Core/include/Acts/Geometry/GeometryIdentifier.hpp	94.59% <ø> (ø)
Core/include/Acts/Surfaces/DiscSurface.hpp	100.00% <ø> (ø)
Core/include/Acts/Surfaces/LineSurface.hpp	100.00% <ø> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[github-actions]

📊: Physics performance monitoring for 36ef813

Full contents

physmon summary

• ✅ Ambisolver orthogonal
• ✅ Ambisolver seeded
• ✅ Ambisolver
• ✅ AMVF orthogonal
• ✅ AMVF seeded
• ✅ AMVF truth_estimated
• ✅ AMVF truth_smeared
• ✅ AMVF ttbar
• ✅ CKF orthogonal
• ✅ CKF seeded
• ✅ CKF truth_estimated
• ✅ CKF truth_smeared
• ✅ CKF ttbar
• ✅ Truth tracking (GSF)
• ✅ IVF orthogonal
• ✅ IVF seeded
• ✅ IVF truth_estimated
• ✅ IVF truth_smeared
• ✅ Particles final fatras
• ✅ Particles final geant4
• ✅ Particles inital fatras
• ✅ Particles inital geant4
• ✅ Seeding orthogonal
• ✅ Seeding seeded
• ✅ Seeding truth_estimated
• ✅ Seeding ttbar
• ✅ Track Summary CKF orthogonal
• ✅ Track Summary CKF seeded
• ✅ Track Summary CKF truth_estimated
• ✅ Track Summary CKF truth_smeared
• ✅ Truth tracking

[felix-russo]

Otherwise good to go

[andiwand]

@paulgessinger if you like we can merge this already and fix the images afterwards since it seems to be an unrelated problem

[paulgessinger]

I'm just doing some more CI debugging

[paulgessinger]

Ok, dead end for right now. Let's merge.

[acts-project-service]

✅ Athena integration test results

✅ All tests successful

status	job
🟢	run_unit_tests
🟢	run_ci_tests: ../athena/AtlasTest/CITest/test/ActsWorkflow.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/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

[paulgessinger]

Failure with a missing symbol. This seems weird, I'm investigating.

[paulgessinger]

Confirmed technical error.