chore!: removing PlanarCluster, PlanarClusterization, IdenficiationModule and…

[asalzburger]

Spring cleaning campaign - part 1

This PR removes the redundant

• Core/Digitization ( replaced by Fatras/Digitization since long ago)
• Plugins/IdentificationModule (only there to add the DigitizationModule and Identifier to detector elements)
• PlanarCluster infrastructure (replaced by Measurement)

Such as the associated Algorithm, helpers and I/O modules in Examples

This is marked BREAKING because it removes a component (albeit unused from Core).

It also moves the Identifier definition into the relevant detector elements.

[asalzburger]

@osbornjd - I tag you on this one, because I think this will have (small) implications for sPHENIX - as you use the TGeo geometry infrastructure.

Botton line, I removed the intermediate IdentifiedDetectorElement in order to get rid of 4k lines of unused code, we still have the same functionality, I just shifted the Identifier into the TGeoElement definition instead.

[github-actions]

📊: Physics performance monitoring for 55def06

Full contents

physmon summary

• ✅ CKF truth_smeared
• ✅ IVF truth_smeared
• ✅ AMVF truth_smeared
• ✅ Track Summary CKF truth_smeared
• ✅ Seeding truth_estimated
• ✅ CKF truth_estimated
• ✅ IVF truth_estimated
• ✅ AMVF truth_estimated
• ✅ Track Summary CKF truth_estimated
• ✅ Seeding seeded
• ✅ CKF seeded
• ✅ IVF seeded
• ✅ AMVF seeded
• ✅ AMVF (+grid seeder) seeded
• ✅ Track Summary CKF seeded
• ✅ Seeding orthogonal
• ✅ CKF orthogonal
• ✅ IVF orthogonal
• ✅ AMVF orthogonal
• ✅ Track Summary CKF orthogonal
• ✅ Ambisolver seeded
• ✅ Ambisolver orthogonal
• ✅ Seeding ttbar
• ✅ CKF ttbar
• ✅ Ambisolver
• ✅ Track Summary CKF ttbar
• ✅ AMVF ttbar
• ✅ AMVF (+grid seeder) ttbar
• ✅ Truth tracking (GSF)
• ✅ Truth tracking
• ✅ Truth tracking (GX2F)
• ✅ Particles fatras
• ✅ Particles geant4

[acts-project-service]

✅ Athena integration test results [6ba0582]

✅ All tests successful

status	job	report
🟢	run_unit_tests
🟢	test_ActsEFTrackFit
🟢	test_ActsPersistifySeeds
🟢	test_ActsBenchmarkWithSpot
🟢	test_ActsAnalogueClustering
🟢	test_ActsConversionWorkflow
🟢	test_ActsWorkflowHeavyIons
🟢	test_ActsWorkflowFastTracking
🟢	test_ActsWorkflowCached
🟢	test_ActsWorkflow
🟢	test_ActsValidateAmbiguityResolution
🟢	test_ActsValidateResolvedTracks
🟢	test_ActsValidateTracks
🟢	test_ActsValidateActsCoreSpacePoints
🟢	test_ActsValidateActsSpacePoints
🟢	test_ActsValidateSeeds
🟢	test_ActsValidateOrthogonalSeeds
🟢	test_ActsValidateClusters
🟢	test_ActsPersistifyEDM
🟢	test_ActsGSFRefitting
🟢	test_ActsKfRefitting
🟢	test_ActsExtrapolationAlgTest
🟢	test_ActsITkTest
🟢	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

[osbornjd]

Thanks for the heads up. I think this actually shouldn't affect us too much as our experiment specific detector element class inherits directly from the TGeoDetectorElement. So it may be completely transparent to us...

[andiwand]

🧹