Skip to content

refactor(traccc): Clean up CCL kernel types - #6080

Merged
stephenswat merged 1 commit into
acts-project:mainfrom
stephenswat:traccc/refactor/ccl_kernel_cleanup
Sep 15, 2026
Merged

stephenswat merged 1 commit into
acts-project:mainfrom
stephenswat:traccc/refactor/ccl_kernel_cleanup

Conversation

@stephenswat

Copy link
Copy Markdown
Member

This commit unifies the CCL kernel types to 32 bit integers, which are most efficient on GPUs. It also removes some unused parameters to make the code more readable.

@github-actions github-actions Bot added this to the next milestone Sep 14, 2026
This commit unifies the CCL kernel types to 32 bit integers, which are
most efficient on GPUs. It also removes some unused parameters to make
the code more readable.
@stephenswat
stephenswat force-pushed the traccc/refactor/ccl_kernel_cleanup branch from 6dcf311 to accef70 Compare September 14, 2026 11:26
@stephenswat

Copy link
Copy Markdown
Member Author

Performance summary

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

Graphical

Tabular

KernelReciprocal ThroughputParallelism
bbbf49d6dcf311Deltabbbf49d6dcf311
propagate_to_next_surface4.19 ms4.19 ms-0.1%4.884.88
find_tracks1.14 ms1.14 ms-0.0%1.841.84
count_doublets787.87 μs788.09 μs0.0%1.611.61
count_triplets552.89 μs553.73 μs0.2%1.021.02
find_doublets517.20 μs508.10 μs-1.8%3.083.08
ccl_kernel442.19 μs440.16 μs-0.5%1.711.71
condense_tracks250.07 μs250.52 μs0.2%5.775.76
find_triplets160.07 μs159.76 μs-0.2%1.311.32
build_tracks125.33 μs125.58 μs0.2%3.723.72
Thrust::sort85.07 μs85.03 μs-0.0%17.7217.73
select_seeds58.36 μs58.50 μs0.2%1.341.34
estimate_track_params26.35 μs26.31 μs-0.2%2.692.68
populate_grid23.50 μs23.54 μs0.2%1.221.22
count_grid_capacities21.90 μs21.88 μs-0.1%1.221.22
remove_duplicates19.84 μs19.94 μs0.5%25.7625.64
update_triplet_weights14.67 μs14.64 μs-0.2%1.271.27
fill_sorted_measurements12.21 μs12.15 μs-0.5%1.501.50
fill_finding_propagation_sort_keys9.84 μs9.84 μs0.1%7.237.23
form_spacepoints7.28 μs7.32 μs0.5%1.471.47
reduce_triplet_counts5.46 μs5.49 μs0.6%3.083.08
fill_measurement_sort_keys3.06 μs3.07 μs0.3%1.481.48
count_spacepoints2.97 μs2.97 μs0.1%1.471.47
unknown1.79 μs1.79 μs0.1%9.859.85
Thrust::scan1.79 μs1.78 μs-0.1%92.3892.61
fill_finding_duplicate_removal_sort_keys1.56 μs1.56 μs0.2%38.1438.09
create_device_detectorNoneNone-0.2%32764.7232764.72
Total8.47 ms8.45 ms-0.1%3.723.72

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.

@github-actions

Copy link
Copy Markdown
Contributor

Public API surface diff

No change to the public API surface. ✅

@sonarqubecloud

Copy link
Copy Markdown

@github-actions

Copy link
Copy Markdown
Contributor

📊: Physics performance monitoring for accef70

Full contents

physmon summary

@paulgessinger paulgessinger left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

looks simple enough.

@stephenswat
stephenswat added this pull request to the merge queue Sep 15, 2026
Merged via the queue into acts-project:main with commit 10b9dd7 Sep 15, 2026
49 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants