[12_4_X] Fix EMTF Run 3 CSC pattern calculation #38320
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
PR description:
In
PrimitiveConversion
we useCreateCSCDetId
to calculate the equivalent Run 2 pattern ID for Run 3 CSC TPs. This was being called before other information about the CSC hit was set, so it was not working properly before. Since Run 3 CSC TPs are not enabled in the emulator yet, this wasn't caught earlier.I moved the code a couple of lines down and now it gets called after we set endcap, station, ring, and chamber informations to the hit. It now functions as it should.
No changes are epxected until we enable Run 3 CSC TPs
PR validation:
Validated with local repository after enabling Run 3 CSC TPs and the run 2 pattern IDs are now set correctly.
if this PR is a backport please specify the original PR and why you need to backport that PR:
backport of #38319