Skip to content

refactor(detray): Split MBF parameter updater into new class - #6051

Open
stephenswat wants to merge 1 commit into
acts-project:mainfrom
stephenswat:detray/refactor/separate_mbf_parameter_updater
Open

refactor(detray): Split MBF parameter updater into new class#6051
stephenswat wants to merge 1 commit into
acts-project:mainfrom
stephenswat:detray/refactor/separate_mbf_parameter_updater

Conversation

@stephenswat

Copy link
Copy Markdown
Member

This commit splits the parameter updater type in two, preserving a non-MBF-enabled version and a new type which supports Jacobian propagation for MBF workflows.

This commit splits the parameter updater type in two, preserving a
non-MBF-enabled version and a new type which supports Jacobian
propagation for MBF workflows.
@stephenswat
stephenswat force-pushed the detray/refactor/separate_mbf_parameter_updater branch from e6a9a20 to cdc392e Compare September 8, 2026 15:43
@stephenswat

Copy link
Copy Markdown
Member Author

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 --conditions-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 298341 to 298342 (+0.0%)

Seeding plots



Track finding performance

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

Finding plots









Track fitting performance

Fitting plots














Seeding to track finding relative performance

Seeding to track finding plots



Note

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

@github-actions github-actions Bot added this to the next milestone Sep 8, 2026
@github-actions github-actions Bot added Component - Detray Affects the Detray project Component - Traccc labels Sep 8, 2026
@stephenswat

Copy link
Copy Markdown
Member Author

Performance summary

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

Graphical

Tabular

KernelReciprocal ThroughputParallelism
8474febcdc392eDelta8474febcdc392e
propagate_to_next_surface4.20 ms4.20 ms0.0%4.884.88
find_tracks1.19 ms1.19 ms0.6%1.841.83
count_doublets792.57 μs789.21 μs-0.4%1.611.61
count_triplets557.73 μs557.16 μs-0.1%1.021.02
find_doublets514.00 μs513.16 μs-0.2%3.083.08
ccl_kernel434.11 μs434.77 μs0.2%1.711.71
Thrust::sort384.57 μs384.32 μs-0.1%7.277.27
condense_tracks250.69 μs250.78 μs0.0%5.765.76
find_triplets161.22 μs160.19 μs-0.6%1.311.31
build_tracks125.53 μs125.37 μs-0.1%3.723.71
select_seeds58.65 μs58.55 μs-0.2%1.341.34
estimate_track_params26.43 μs26.35 μs-0.3%2.682.69
populate_grid23.59 μs23.56 μs-0.1%1.221.22
count_grid_capacities21.91 μs21.95 μs0.2%1.221.22
remove_duplicates20.00 μs19.84 μs-0.8%25.6325.77
fill_sorted_measurements16.18 μs16.08 μs-0.6%1.131.13
update_triplet_weights14.74 μs14.64 μs-0.7%1.271.27
fill_finding_propagation_sort_keys9.85 μs9.84 μs-0.1%7.227.23
form_spacepoints7.44 μs7.43 μs-0.1%1.471.48
reduce_triplet_counts5.50 μs5.50 μs-0.1%3.083.08
unknown5.05 μs5.04 μs-0.2%4.274.27
count_spacepoints2.99 μs2.99 μs-0.1%1.471.47
Thrust::scan1.79 μs1.78 μs-0.4%92.6992.82
fill_finding_duplicate_removal_sort_keys1.55 μs1.55 μs-0.1%38.1338.14
create_device_detectorNoneNone0.1%32764.7232764.72
Total8.82 ms8.82 ms0.0%3.733.73

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.

@sonarqubecloud

sonarqubecloud Bot commented Sep 8, 2026

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

Public API surface diff

No change to the public API surface. ✅

@github-actions

github-actions Bot commented Sep 8, 2026

Copy link
Copy Markdown
Contributor

📊: Physics performance monitoring for cdc392e

Full contents

physmon summary

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant