Adding first LHE sequence to RelVal #6554
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…adgraph_LHE_13TeV_cff.py as first step for LHE generation in RelVal. 11/17/2014 (nugent@physik.rwth-aachen.de)
…f.py to convert lhe to hep format. (first version - issue with missing LHERunInfoProduct)
…in relval generator sequence... + added to generator sequence
…or this pull request + patches to lhe relval found during testing(validation in progress - may have more).
…o new files and adding originals back. Adding patch to new Hadronizer_MgmMatchTuneCUETP8M1JetMatching*. Modifying relval_steps.py for this update.
…nitialised data members, read run number from configuration.
…iguration, not standard input.
…tching* files which was not removed
…the DQM file when running 2 spteps before harvesting. <nugent@physik.rwth-aachen.de>
…line validation for cosmics
|
|
my tests fail because of incorrectly configured cvmfs, thanks Josh for pointing that out. Will re-do. |
…run1_mc to auto:run2_mc_FULL; 2) switch to auto:run2_mc_Full also HARVESTGEN; 3) change HARVESTGEN2 to steps['HARVESTGEN2']=merge([{'--filein':'file:step2_inDQM.root'},steps['HARVESTGEN']]) + setting q cut to 20GeV at Josh's request
… from-CMSSW_7_3_X_2014-11-25-1400 Conflicts: Configuration/PyReleaseValidation/python/relval_steps.py
Update RelVal so length of step1LHEDefault step is ~3-4hr using estimate of 2sec per event.
|
This can be closed. (replaced by #6626) |
cmsbuild
added
alca-pending
analysis-pending
comparison-pending
db-pending
dqm-pending
generators-pending
geometry-pending
hlt-pending
l1-pending
orp-pending
pdmv-pending
pending-signatures
reconstruction-pending
tests-pending
and removed
comparison-pending
generators-pending
orp-pending
pdmv-pending
pending-signatures
tests-approved
labels
|
Pull request #6554 was updated. @diguida, @ianna, @cerminar, @Martin-Grunewald, @covarell, @rcastello, @bendavid, @perrotta, @civanch, @franzoni, @cmsbuild, @fwyzard, @ktf, @vciulli, @Dr15Jones, @mdhildreth, @deguio, @slava77, @ggovi, @vadler, @mmusich, @mulhearn, @apfeiffer1, @nclopezo, @danduggan, @boudoul, @thuer, @monttj, @srimanob, @StoyanStoynev can you please check and sign again. |
|
closing 7_3_X pr since it is in 6626 for 7_4_x. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request include the first RelVal workflows with LHE for DY->ll+Jets. Moreover, following the request of Josh, the jet matching has been changed to use the new pythia8 jet matching for the LHE workflows. All of RelVal sequences that use LHE files in the generator and extended generator workflows have had the a fragment added for easy use in the future. The full generator and extended RelVal workflows have been tested on a small number of events.