-
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
Validation #2833
Validation #2833
Conversation
A new Pull Request was created by @kasmi (Azeddine Kasmi) for CMSSW_6_2_X_SLHC. Validation It involves the following packages: Validation/CaloTowers @ojeda, @danduggan, @rovere, @cmsbuild, @nclopezo, @deguio, @Degano can you please review it and eventually sign? Thanks. |
+1 From: cmsbuild [[email protected]] A new Pull Request was created by @kasmihttps://github.com/kasmi (Azeddine Kasmi) for CMSSW_6_2_X_SLHC. Validation It involves the following packages: Validation/CaloTowers @ojedahttps://github.com/ojeda, @dandugganhttps://github.com/danduggan, @roverehttps://github.com/rovere, @cmsbuildhttps://github.com/cmsbuild, @nclopezohttps://github.com/nclopezo, @deguiohttps://github.com/deguio, @deganohttps://github.com/degano can you please review it and eventually sign? Thanks. — |
There is a segmentation violation in HcalRecHitsClient:hcalrechitsClient@endRun for runTheMatrix -w upgrade -l 10000,10200,11600,13000 This is the relevent portions of the call stack
Event before this there are some errors of this sort which might need to be sorted out
|
merge With the updates tests exhibit only known failures. |
@@ -33,13 +33,13 @@ | |||
//dbe_->setCurrentFolder(dirName_); | |||
dbe_->setCurrentFolder("HcalDigisV/HcalDigiTask"); | |||
|
|||
// false for regular relval and true for SLHC relval |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change has reintroduced a bug fixed by #2483. doSLHC_ must be set before the call to booking(...).
This has already been merged so I'll put in another pull request to fix it.
Some fixes to the Validation package for HcalRecHits and HcalDigis plots.