Skip to content
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

Mini aodvalidation: V0 #11925

Merged
merged 4 commits into from Dec 3, 2015

Conversation

cmsbuild
Copy link
Contributor

A new analyzer has been deployed in Validation/RecoEgamma and a new sequence to be included in Configuration/StandardSequences/python/Validation_cff.py (also committed). The machinery is meant only for MC miniAOD. It has all been tested with "central" running using runTheMatrix.py --what standard -i all -l 1332 --command=' -n 200'
Automatically ported from CMSSW_7_6_X #11750 (original by @nancymarinelli).

@cmsbuild
Copy link
Contributor Author

A new Pull Request was created by @cmsbuild for CMSSW_8_0_X.

Mini aodvalidation: V0

It involves the following packages:

Configuration/StandardSequences
Validation/RecoEgamma

@danduggan, @cmsbuild, @franzoni, @deguio, @vanbesien, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @richard-cms, @cerati, @dgulhan this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@deguio
Copy link
Contributor

deguio commented Nov 10, 2015

+1
ok, we discussed. new dev will come with a different PR.

@cmsbuild
Copy link
Contributor Author

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-any-integration/9604/console

@cmsbuild
Copy link
Contributor Author

@cmsbuild
Copy link
Contributor Author

davidlange6 added a commit that referenced this pull request Dec 3, 2015
@davidlange6 davidlange6 merged commit f324ab8 into cms-sw:CMSSW_8_0_X Dec 3, 2015
@deguio
Copy link
Contributor

deguio commented Dec 5, 2015

ciao david,
sorry I cannot have a look before monday. feel free to revert this PR.
thanks for reporting,
F.

@nancymarinelli
Copy link
Contributor

Hi Dave,
I am looking at this now. I am trying with the WF 1332. runTheMatrix --
what i 1332.

Are you sure that in the sequence below the DQM was run ?
From the labels of the log that is not clear.

Nancy

On 04/12/15 14:36, David Lange wrote:

@nancymarinelli https://github.com/nancymarinelli @deguio
https://github.com/deguio - it looks like this PR is introducing
crashes in the integration builds. Could you have a look today?

https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc493/CMSSW_8_0_X_2015-12-03-2300/pyRelValMatrixLogs/run/19.0_SingleGammaPt35+SingleGammaPt35INPUT+DIGI+RECO+HARVEST/step3_SingleGammaPt35+SingleGammaPt35INPUT+DIGI+RECO+HARVEST.log


Reply to this email directly or view it on GitHub
#11925 (comment).


Nancy Marinelli
Research Associate Professor
University of Notre Dame, IN, US
CERN, Bdg 40/3-A01, 1211 Geneva
SWITZERLAND
Phone +41-22-76-70809
fax +41-22-76-78940

@davidlange6
Copy link
Contributor

Hi @nancymarinelli
@Dr15Jones fixed this issue on Friday - Cheers

On Dec 7, 2015, at 12:53 PM, nancymarinelli notifications@github.com wrote:

Hi Dave,
I am looking at this now. I am trying with the WF 1332. runTheMatrix --
what i 1332.

Are you sure that in the sequence below the DQM was run ?
From the labels of the log that is not clear.

Nancy

On 04/12/15 14:36, David Lange wrote:

@nancymarinelli https://github.com/nancymarinelli @deguio
https://github.com/deguio - it looks like this PR is introducing
crashes in the integration builds. Could you have a look today?

https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc493/CMSSW_8_0_X_2015-12-03-2300/pyRelValMatrixLogs/run/19.0_SingleGammaPt35+SingleGammaPt35INPUT+DIGI+RECO+HARVEST/step3_SingleGammaPt35+SingleGammaPt35INPUT+DIGI+RECO+HARVEST.log


Reply to this email directly or view it on GitHub
#11925 (comment).


Nancy Marinelli
Research Associate Professor
University of Notre Dame, IN, US
CERN, Bdg 40/3-A01, 1211 Geneva
SWITZERLAND
Phone +41-22-76-70809
fax +41-22-76-78940

Reply to this email directly or view it on GitHub.

@nancymarinelli
Copy link
Contributor

oh, good to know. What was it by the way ? Something related to my piece
of software or something else ?

Nancy

On 07/12/15 13:08, David Lange wrote:

Hi @nancymarinelli
@Dr15Jones fixed this issue on Friday - Cheers

On Dec 7, 2015, at 12:53 PM, nancymarinelli
notifications@github.com wrote:

Hi Dave,
I am looking at this now. I am trying with the WF 1332. runTheMatrix --
what i 1332.

Are you sure that in the sequence below the DQM was run ?
From the labels of the log that is not clear.

Nancy

On 04/12/15 14:36, David Lange wrote:

@nancymarinelli https://github.com/nancymarinelli @deguio
https://github.com/deguio - it looks like this PR is introducing
crashes in the integration builds. Could you have a look today?

https://cmssdt.cern.ch/SDT/cgi-bin/buildlogs/slc6_amd64_gcc493/CMSSW_8_0_X_2015-12-03-2300/pyRelValMatrixLogs/run/19.0_SingleGammaPt35+SingleGammaPt35INPUT+DIGI+RECO+HARVEST/step3_SingleGammaPt35+SingleGammaPt35INPUT+DIGI+RECO+HARVEST.log


Reply to this email directly or view it on GitHub
#11925 (comment).


Nancy Marinelli
Research Associate Professor
University of Notre Dame, IN, US
CERN, Bdg 40/3-A01, 1211 Geneva
SWITZERLAND
Phone +41-22-76-70809
fax +41-22-76-78940

Reply to this email directly or view it on GitHub.


Reply to this email directly or view it on GitHub
#11925 (comment).


Nancy Marinelli
Research Associate Professor
University of Notre Dame, IN, US
CERN, Bdg 40/3-A01, 1211 Geneva
SWITZERLAND
Phone +41-22-76-70809
fax +41-22-76-78940

@davidlange6
Copy link
Contributor

This was the fix: #12670

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants