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
Diamond Sampic reco #34759
Diamond Sampic reco #34759
Conversation
@ChrisMisan, CMSSW_12_1_X branch is closed for direct updates. cms-bot is going to move this PR to master branch. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34759/24424 |
A new Pull Request was created by @ChrisMisan (Christopher) for master. It involves the following packages:
@perrotta, @jpata, @cmsbuild, @slava77 can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-34759/24592
|
@cmsbuild please test |
@cmsbuild please test |
-1 Failed Tests: RelVals-INPUT RelVals-INPUT
Expand to see more relval errors ...Comparison SummarySummary:
|
Sorry @jpata, what is the cause of error here? |
@cmsbuild please test it's not related to this PR, something seems wrong in the test infrastructure since some time. let's try again just in case. |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-e2f6ce/17731/summary.html Comparison SummarySummary:
|
+reconstruction
|
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. @perrotta, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2) |
+1 |
-----------------------------General overview----------------------------
Goal of this PR is to extend and modify existing sampic reco source(under RecoPPS/Local/) to provide support for the diamond detectors. Aside from providing additional functionality, the changes were structured in a way that avoid any code duplication and provides full backward compatibility. Users can choose between the two flows by using diamondSampicLocalReconstruction or totemTimingLocalReconstruction in a process path.
To test those changes, an additional producer named DiamondSampicDigiProducer was created. This producer takes testbeam data and converts them to totemTimingDigi (by utilising fake channel mapping), which could be then passed to the reco stage. PR contains an appropriate test file(RecoPPS/Local/test/totemTimingDigiConverter_reco_cfg) which produces reco by using this module as a starting point.
-----------------------------Implementation----------------------------
Changes to the existing file mainly concern source generalisation-to support both UFSD and DIAMOND Sampic flows. This is done either through class templates and then class specialization or extracting geometry information and applying conditional statements. Aside from those changes, some previously hardcoded parameters were extracted to python config and the obsolete code was removed.
For a more detailed overview, please refer to the provided pdf https://cernbox.cern.ch/index.php/s/2RQ0hD5B0QjLYYA