-
Notifications
You must be signed in to change notification settings - Fork 25
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
O2O Updates #400
Comments
This has been tested online with CMSSW_8_0_15 and the O2O machinery works. (details to follow). |
Verified all payloads: L1TUtmTriggerMenu - no change wrt. previous version (as expected) L1TGlobalPrescalesVetos - no change (as expected) L1TMuonOverlapParams - no change (as expected) L1TMuonEndcapParams - changed as expected, not yet in use anywhere L1TMuonBarrelParams - changed, waiting for Giannis to validate it CaloParams:
|
After the cms-sw#15204 PR will be merged, I propose to go forward with two new PRs for my new fullL1TO2Ofor80X and fullL1TO2Ofor81X branches |
Although, before PRing we can also straighten out some "private" solutions bypassing the need to change CondFormats. For example one of these home-breed solutions: https://github.com/kkotov/cmssw/blob/fullL1TO2Ofor80X/L1Trigger/L1TMuon/interface/L1TMuonGlobalParams_PUBLIC.h do nothing else, but simply substitutes "private" for "public" in https://github.com/kkotov/cmssw/blob/fullL1TO2Ofor80X/CondFormats/L1TObjects/interface/L1TMuonGlobalParams.h#L215 |
The first pass PRs which includes many updates (even since Stage 1) and the Stage 2 L1 menu O2O is here:
The next pass PR will be based (for 80x) this branch in work:
(Includes everything including Calo Layer2 and capitalization fixes for parser, which requires some updates)
The text was updated successfully, but these errors were encountered: