-
Notifications
You must be signed in to change notification settings - Fork 3
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
Uodated GEM-CSC triager LUTs #13
Conversation
preliminary reduction and renaming of tables
new nomenclature and updated tables with increased bit length
A new Pull Request was created by @giovanni-mocellin for branch master. @smuzaffar, @epalencia, @aandvalenzuela, @iarspider, @cmsbuild, @rekovic, @cecilecaillol can you please review it and eventually sign? Thanks. |
@cecilecaillol I believe you'd need to sign this too |
+l1 |
please test |
-1 Failed Tests: UnitTests RelVals AddOn The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Unit TestsI found errors in the following unit tests: ---> test TestSubProcess had ERRORS ---> test testPhase2PixelNtuple had ERRORS RelVals
Expand to see more relval errors ...AddOn Tests
Expand to see more addon errors ... |
@cmsbuild please test with cms-sw/cmssw#38373 |
-1 Failed Tests: RelVals The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: RelVals
|
@cmsbuild please test with cms-sw/cmssw#38373 |
-1 Failed Tests: RelVals RelVals
|
@cmsbuild please test with cms-sw/cmssw#38373 and cms-sw/cmssw#38775 |
merge |
This PR contains changes to the GEM-CSC trigger lookup tables to match the ones used in the CSC OTMB FW.
The main changes are:
This PR is linked to a PR to be submitted to CMSSW master, which uses these LUT files. A link to this PR will be added to the description of the parallel one in CMSSW.