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.

Sign up for GitHub

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

Jump to bottom

Changes needed for SUSY parameter scans (80x) #13684

Merged
merged 25 commits into from
Mar 17, 2016
Merged

Changes needed for SUSY parameter scans (80x) #13684

merged 25 commits into from
Mar 17, 2016

Conversation

bendavid
Copy link
Contributor

Backport of #13530 for SUSY fastsim production in CMSSW 80x.
(With a strong desire to run some test workflows through mcm+central production sooner than later to discover any unexpected issues with the workflow.)

@bendavid
initial implementation of pythia config randomization
1eb6498
@bendavid
workaround for random number generatoin in beginLumi and add example
718f329
@bendavid
update resetting of random number seed so that it only relies on the …
e84d0ed 
…internal state of the random engine and not the current seed
@bendavid
re-arrange use of random numbers
c398eed
@bendavid
clean up inheritance of pythia8 interface
72e477b
@bendavid
fully re-initialize pythia8 (delete and reinstantiate main object) to…
26d9bf5 
… avoid potential issues with slha double initialization
@bendavid
add randomized config information to GenEventInfoProduct
14ea9ce
@bendavid
extend random config storage to GenLumiInfoProduct plus a few fixes
f47f7d7
@bendavid
make ConfigDescription optional
c7f734e
@bendavid
update examples
e4f1593
@bendavid
further fix to examples
4328fae
@bendavid
add full list of userhooks also for internal generation
ff1e169
@bendavid
remove debug printout
283280c
@bendavid
further updates to random seed initialization and initial support for…
c6c9bd0 
… LHE generation at beginLuminosityBlock in the hadronizers
@bendavid
automatically determine number of events per lumi section from source…
085a06b 
… configuration
@bendavid
revert DataFormats changes and add new GenLumiInfoHeader which is pro…
ccfd989 
…duced in beginLuminosityBlockProduce, plus fix for gridpack randomization
@bendavid
enforce strict error handling in lhe production script
e8f7878
@bendavid
update examples and add example for madgraph parameter scan
32850b7
@bendavid
add GenLumiInfoHeader to relevant event content definitions
f2009c2
@bendavid
fill weight information in events
3e52c35
@bendavid
add lhe header and weight name information to GenLumiInfoHeader
a450577
@bendavid
populate additional lhe information within limitations of current pyt…
b313631 
…hia8 lhe parser
@bendavid
restore lhe cleanup
b96989f
@bendavid
check for missing lhe init information
90e1fd7
@bendavid
remove debug printout
06c3c39
@cmsbuild cmsbuild added this to the Next CMSSW_8_0_X milestone Mar 10, 2016
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @bendavid (Josh Bendavid) for CMSSW_8_0_X.

It involves the following packages:

GeneratorInterface/Configuration
GeneratorInterface/Core
GeneratorInterface/LHEInterface
GeneratorInterface/Pythia8Interface
PhysicsTools/PatAlgos
SimDataFormats/GeneratorProducts

@vciulli, @covarell, @cvuosalo, @thuer, @monttj, @cmsbuild, @bendavid, @slava77, @vadler, @davidlange6, @govoni can you please review it and eventually sign? Thanks.
@rappoccio, @mkirsano, @imarches, @ahinzmann, @acaudron, @mmarionncern, @TaiSakuma, @jdolen, @nhanvtran, @agrohsje, @schoef, @ferencek, @gpetruc, @mariadalfonso, @pvmulder this is something you requested to watch as well.
@slava77, @Degano, @smuzaffar you are the release manager for this.

cms-bot commands are list here #13028

@slava77
Copy link
Contributor

slava77 commented Mar 10, 2016

@cmsbuild please test

@cmsbuild
Copy link
Contributor

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

@cmsbuild
Copy link
Contributor

+1
Tested at: 06c3c39
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13684/11829/summary.html

@cmsbuild
Copy link
Contributor

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-13684/11829/summary.html

@covarell
Copy link
Contributor

+1

@slava77
Copy link
Contributor

slava77 commented Mar 12, 2016

+1

for #13684 06c3c39

@covarell
Copy link
Contributor

Can this be merged? We have another 81X/80X development which should be re-based after this change. Thanks

davidlange6 added a commit that referenced this pull request Mar 17, 2016
@davidlange6
Merge pull request #13684 from bendavid/pythia8cfgrandom_80xV4
ddf0521 
Changes needed for SUSY parameter scans (80x)
@davidlange6 davidlange6 merged commit ddf0521 into cms-sw:CMSSW_8_0_X Mar 17, 2016
@covarell covarell mentioned this pull request Apr 18, 2016
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
analysis-pending comparison-available generators-approved orp-pending pending-signatures reconstruction-approved tests-approved
Projects
None yet
Milestone
CMSSW_8_0_X
Development

Successfully merging this pull request may close these issues.
5 participants
@bendavid @cmsbuild @slava77 @covarell @davidlange6