Skip to content

Conversation

@niermann999
Copy link
Contributor

@niermann999 niermann999 commented Nov 21, 2025

Build the ODD and ITk detectors with their custom metadata. Also remove the io roundtrip in the tests for the telescope and toy detector, since the custom metadata of the test detectors is already getting compiled and they do not need to be read into the default detector anymore

@niermann999 niermann999 added the improvement Improve an existing feature label Nov 21, 2025
@niermann999 niermann999 force-pushed the feat-detector-metadata branch 3 times, most recently from 465458d to 79a6cca Compare November 21, 2025 19:09
@niermann999 niermann999 force-pushed the feat-detector-metadata branch from 79a6cca to 921e7e9 Compare November 21, 2025 19:52
@niermann999 niermann999 force-pushed the feat-detector-metadata branch from 921e7e9 to 302ffa8 Compare November 26, 2025 16:41
@niermann999 niermann999 added the cleanup Makes the code all clean and tidy label Nov 26, 2025
@niermann999 niermann999 force-pushed the feat-detector-metadata branch 2 times, most recently from 48179f6 to b6a1e3f Compare November 27, 2025 13:34
@stephenswat

This comment was marked as outdated.

@stephenswat

This comment was marked as outdated.

Copy link
Member

@krasznaa krasznaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to make your life more difficult.

@krasznaa
Copy link
Member

krasznaa commented Dec 3, 2025

The CMake part I was able to update pretty quickly. But the code is unfortunately quite broken in the PR at the moment. 😦 The changes in the tests will have to be reverted. Or the tests will need to be much more re-written.

I can show you the details tomorrow, but basically, the toy detector and telescope fitting tests deeply assume at the moment that they would use the same detector type. As the detector type is defined in a common base class to these tests. So the unit tests are just crashing in this PR at the moment. 😦

Again, to be discussed tomorrow...

@krasznaa
Copy link
Member

krasznaa commented Dec 3, 2025

You can find my CMake changes here: krasznaa@6103b5e

@niermann999
Copy link
Contributor Author

You can find my CMake changes here: krasznaa@6103b5e

Thank you, I will have have look

@niermann999 niermann999 force-pushed the feat-detector-metadata branch 2 times, most recently from 52a07b7 to 3c06390 Compare December 4, 2025 10:32
@stephenswat

This comment was marked as outdated.

@niermann999 niermann999 force-pushed the feat-detector-metadata branch from 3c06390 to 0ff07ee Compare December 4, 2025 12:53
@stephenswat

This comment was marked as outdated.

@niermann999 niermann999 force-pushed the feat-detector-metadata branch 2 times, most recently from 254521f to 0d8b0ee Compare December 4, 2025 15:42
@niermann999 niermann999 force-pushed the feat-detector-metadata branch from 0d8b0ee to 90de67f Compare December 5, 2025 15:36
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 5, 2025

@stephenswat
Copy link
Member

Performance summary

Here is a summary of the performance effects of this PR:

Graphical

Tabular

KernelReciprocal ThroughputParallelism
8e3edfd90de67fDelta8e3edfd90de67f
propagate_to_next_surface8.40 ms6.25 ms-25.6%3.484.27
fit_forward2.04 ms1.92 ms-6.0%7.347.34
find_tracks1.38 ms1.38 ms-0.1%1.811.81
ccl_kernel823.77 μs823.42 μs-0.0%1.371.37
fit_backward1.10 ms670.36 μs-39.1%5.505.49
count_doublets622.61 μs613.53 μs-1.5%1.611.61
count_triplets588.96 μs588.60 μs-0.1%1.021.02
find_doublets444.54 μs445.88 μs0.3%3.083.08
Thrust::sort384.92 μs384.77 μs-0.0%7.367.36
find_triplets173.65 μs172.24 μs-0.8%1.311.31
select_seeds53.04 μs53.79 μs1.4%1.341.34
remove_duplicates23.67 μs23.68 μs0.1%26.0526.01
populate_grid23.49 μs23.36 μs-0.5%1.221.22
count_grid_capacities22.21 μs22.12 μs-0.4%1.221.22
fill_sorted_measurements19.74 μs19.70 μs-0.2%1.131.13
update_triplet_weights14.96 μs14.97 μs0.1%1.271.27
estimate_track_params14.26 μs14.26 μs-0.0%2.152.15
apply_interaction14.25 μs13.96 μs-2.0%6.726.65
fit_prelude9.82 μs9.82 μs-0.0%21.7821.78
fill_finding_propagation_sort_keys8.86 μs8.83 μs-0.3%7.607.62
form_spacepoints7.94 μs8.23 μs3.7%1.481.48
reduce_triplet_counts6.28 μs6.27 μs-0.2%3.083.08
unknown5.04 μs5.05 μs0.3%4.284.27
build_tracks4.10 μs4.09 μs-0.2%14.7814.78
fill_finding_duplicate_removal_sort_keys1.57 μs1.57 μs0.1%37.9737.95
fill_prefix_sum171.95 ns171.97 ns0.0%341.30341.30
fill_fitting_sort_keys153.36 ns152.68 ns-0.4%22.0822.08
Total16.20 ms13.48 ms-16.7%3.794.12

Important

All metrics in this report are given as reciprocal throughput, not as wallclock runtime.

Note

This is an automated message produced upon the explicit request of a human being.

@niermann999 niermann999 requested a review from krasznaa December 5, 2025 15:59
Copy link
Member

@krasznaa krasznaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As long as all the latest CI tests succeed...

@stephenswat
Copy link
Member

Physics performance summary

Here is a summary of the physics performance effects of this PR. Command used:

traccc_seeding_example_cuda --input-directory=/data/Acts/odd-simulations-20240506/geant4_ttbar_mu200 --digitization-file=geometries/odd/odd-digi-geometric-config.json --detector-file=geometries/odd/odd-detray_geometry_detray.json --grid-file=geometries/odd/odd-detray_surface_grids_detray.json --material-file=geometries/odd/odd-detray_material_detray.json --input-events=10 --use-acts-geom-source=on --check-performance --truth-finding-min-track-candidates=5 --truth-finding-min-pt=1.0 --truth-finding-min-z=-150 --truth-finding-max-z=150 --truth-finding-max-r=10 --seed-matching-ratio=0.99 --track-matching-ratio=0.5 --track-candidates-range=5:100 --seedfinder-vertex-range=-150:150

Seeding performance

Total number of seeds went from 298345 to 298342 (-0.0%)

Seeding plots



Track finding performance

Total number of found tracks went from 50184 to 50186 (+0.0%)

Finding plots









Track fitting performance

Fitting plots





















Seeding to track finding relative performance

Seeding to track finding plots



Track finding to track fitting relative performance

Track finding to track fitting plots



Note

This is an automated message produced on the explicit request of a human being.

@stephenswat stephenswat merged commit 8ef4b18 into acts-project:main Dec 5, 2025
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cleanup Makes the code all clean and tidy improvement Improve an existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants