-
Notifications
You must be signed in to change notification settings - Fork 12
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
Adding ZDC LUT for use with L1TZDC Emulator; to be pushed to cms-data… #23
Conversation
…; PR to cms-l1t-iffline already in progress
A new Pull Request was created by @cfmcginn (Chris) for branch master. @smuzaffar, @epalencia, @aandvalenzuela, @iarspider, @cmsbuild, @aloeliger can you please review it and eventually sign? Thanks. |
please test |
-1 Failed Tests: RelVals AddOn RelValsAddOn Tests
Expand to see more addon errors ...
|
@cfmcginn According to your email today, it seems like externals really aren't actually the place for this. We need to instead discuss with the o2o/db experts. |
After further discussion, perhaps this is the correct location for this for the time being, the relative rate of change of this LUT may be such that having it in the externals for testing while a db solution is created is desirable. |
@smuzaffar The test failures are complaining largely about |
Understood - please let me know if these test failures require changes by me, will iterate. |
test parameters:
|
please test
I guess it needs cmssw PR to run properly ? |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c59d2f/34568/summary.html 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: Comparison SummarySummary:
|
+l1 |
+1 |
merge |
LUT for L1TZDCProducer, emulator of ZDC l1 trigger for 2023 HI datataking - PR in progress to cmssw here:
cms-sw/cmssw#42634
Simultaneous PR to cms-l1t-offline/L1Trigger-L1TCalorimeter here:
cms-l1t-offline#20
Tagging a few people to track - @ginnocen @bundocka @aloeliger