-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Define new CLCT digi format for Run-3 #28604
Define new CLCT digi format for Run-3 #28604
Conversation
The code-checks are being triggered in jenkins. |
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-28604/13117
|
A new Pull Request was created by @dildick (Sven Dildick) for master. It involves the following packages: DataFormats/CSCDigi @cmsbuild, @civanch, @mdhildreth can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
please test |
The tests are being triggered in jenkins. |
Comparison job queued. |
Comparison is ready Comparison Summary:
|
@civanch Could you wait with signing off for now? |
@dildick are you expect further development or validation? |
hold pending a possible alternative proposal by @dildick |
Pull request has been put on hold by @fabiocos |
@dildick shall we continue to wait for an update of this PR? |
Hi Silvio, I'm working on an alternative proposal where I expand the CLCT data format for Run-3. I plan to present it soon in a CSC DPG meeting. So I suppose we can close this for now. |
Ok, let's close this PR. Let me know in case you want to open it again. |
PR description:
During Run-3 the CSC DPG group is planning to run the CSC trigger with new CLCT patterns based on a 12-bit comparator code. These new CLCTs will be of type
CSCCLCTRun3Digi
to differentiate with the legacyCSCCLCTDigi
used in Run-1 and Run-2. An introduction to the new CLCT patterns can be found here: https://indico.cern.ch/event/778315/contributions/3238278/subcontributions/268636/attachments/1770627/2876988/181213-CCStatus_WNASH.pdf. A related PR is #28600.PR validation:
The code compiles.
CSCCLCTRun3Digis
are not yet produced in CMSSW. This will be done in a future PR. There should be no change in any standard sequence.if this PR is a backport please specify the original PR:
N/A
@ptcox @tahuang1991