-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
updated the HCAL logical map code to reflect the change in HO ring-0 int... #1081
Conversation
…interface cards, installed during LS1
A new Pull Request was created by @weihua19900704 for CMSSW_7_0_X. updated the HCAL logical map code to reflect the change in HO ring-0 int... It involves the following packages: CondFormats/HcalObjects @smuzaffar, @apfeiffer1, @nclopezo, @demattia, @rcastello, @ggovi can you please review it and eventually sign? Thanks. |
+1 |
@nclopezo David, one of the Jenkins integration tests for this pull request seems to have been running for 21 hours |
Hi Chris, which of the jobs that are here do you mean? https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc481/ the tests for this one finished yesterday and took 34 minutes according to what I see here: https://cmssdt.cern.ch/jenkins/job/Pull-Request-Integration/ARCHITECTURE=slc5_amd64_gcc481/895/ |
My fault, I misread the 21 hours since success as 21 hours of running |
+1 |
This pull request is fully signed and it will be integrated in one of the next IBs unless changes or unless it breaks tests. @ktf can you please take care of it? |
Is this really meant for 70X??? |
updated the HCAL logical map code to reflect the change in HO ring-0 int...
...erface cards, installed during LS1