-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
RecHit map and CaloParticle-LayerCluster associator #29844
Conversation
Comparison is ready Comparison Summary:
|
+1 |
+upgrade some improvements (e.g. use of |
+1 |
Do you have any comments? core: @Dr15Jones @makortel @smuzaffar |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
PR description:
This PR introduces two products (a
std::map
and an associator) which can be consumed by several analyzers, removing code duplication. The first replacement is made for theHGVHistoProducerAlgo::fill_generic_cluster_histos
method, no changes are expected in the output.PR validation:
runTheMatrix -l limited