update of "DisplacedSUSY" relvals for TRK and EXO validation of displaced tracks, for Run 3 and Phase 2 #31692

jalimena · 2020-10-07T08:39:15Z

PR description:

This PR updates the "DisplacedSUSY" relvals, for tracking and exotica validation of displaced tracks.

The DisplacedSUSY process simulates the pair-production of long-lived stops that each decay through an RPV vertex to a lepton (e, mu, or tau with 1/3 probability each) and a b-quark. This sample thus produces displaced leptons.

In this PR, there are a few changes and additions:

The stop mass was updated from 300 GeV to 800 GeV, in order to use a mass that has not yet been excluded. The stop lifetime was updated from 1000mm to 500mm in order to increase the tracker acceptance for displaced tracks. (The "DisplacedSUSY_stopToB_M_300_1000mm_13" gen fragment and WFs were removed and replaced by "DisplacedSUSY_stopToB_M_800_500mm_13" ones.)
In addition to the 13 TeV WFs, 14 TeV WFs were added, in order to validate changes for Run 3 and Phase 2.

This sample was agreed upon in https://its.cern.ch/jira/browse/PDMVRELVALS-96.

I believe the most interesting WFs will be:

23324.0 (2026D49, 14 TeV, no PU)
23524.0 (2026D49, 14 TeV, PU)
11724.0 (2021, 14 TeV, no PU)
11924.0 (2021, 14 TeV, PU)

PR validation:

This PR passes the basic unit tests and code quality checks. I have also successfully tested WF 23285.0 locally.

if this PR is a backport please specify the original PR and why you need to backport that PR:

not a backport

FYI @mtosi @vmariani @elusiani @Fernance @lowette @nsaoulid @kdipetri @mcitron @jandrea @blochd @mmasciov @chayanit @srappocc

…opToB_M_800_500mm_13/14 gen fragments and in WFs

cmsbuild · 2020-10-07T08:39:40Z

The code-checks are being triggered in jenkins.

cmsbuild · 2020-10-07T08:46:21Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-31692/18840

This PR adds an extra 84KB to repository
There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/python/relval_steps.py modified in PR(s): Nano gen dqm #31591
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Integrating VectorHits reconstruction for the Phase2 OT - reprise #31314, Removal of phase2 Tracker geometries T17, T19, T20 and migrations #31673

cmsbuild · 2020-10-07T08:46:46Z

A new Pull Request was created by @jalimena (Juliette Alimena) for master.

It involves the following packages:

Configuration/Generator
Configuration/PyReleaseValidation

@SiewYan, @mkirsano, @jordan-martins, @chayanit, @wajidalikhan, @kpedro88, @cmsbuild, @GurpreetSinghChahal, @agrohsje, @alberto-sanchez can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @makortel, @fabiocos, @slomeo this is something you requested to watch as well.
@silviodonato, @dpiparo, @qliphy you are the release manager for this.

cms-bot commands are listed here

chayanit · 2020-10-07T09:59:43Z

Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py

@@ -1144,7 +1144,8 @@ def __init__(self, howMuch, dataset):
    ('HSCPstop_M_200_TuneCUETP8M1_13TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'HSCPstop_M_200_13')),
    ('RSGravitonToGammaGamma_kMpl01_M_3000_TuneCUETP8M1_13TeV_pythia8_cfi', UpgradeFragment(Kby(9,50),'RSGravitonToGaGa_13')),
    ('WprimeToENu_M-2000_TuneCUETP8M1_13TeV-pythia8_cff', UpgradeFragment(Kby(9,50),'WpToENu_M-2000_13')),
-    ('DisplacedSUSY_stopToBottom_M_300_1000mm_TuneCUETP8M1_13TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'DisplacedSUSY_stopToB_M_300_1000mm_13')),
+    ('DisplacedSUSY_stopToBottom_M_800_500mm_TuneCP5_13TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'DisplacedSUSY_stopToB_M_800_500mm_13')),
+    ('DisplacedSUSY_stopToBottom_M_800_500mm_TuneCP5_14TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'DisplacedSUSY_stopToB_M_800_500mm_14')),


To not ruin the order of existing workflow IDs, please move the new one to the bottom. Thanks.

ok thanks, should be done now

…g workflow IDs

cmsbuild · 2020-10-07T10:31:48Z

The code-checks are being triggered in jenkins.

cmsbuild · 2020-10-07T10:36:31Z

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-31692/18842

This PR adds an extra 84KB to repository
There are other open Pull requests which might conflict with changes you have proposed:
- File Configuration/PyReleaseValidation/python/relval_steps.py modified in PR(s): Nano gen dqm #31591
- File Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py modified in PR(s): Integrating VectorHits reconstruction for the Phase2 OT - reprise #31314, Removal of phase2 Tracker geometries T17, T19, T20 and migrations #31673

cmsbuild · 2020-10-07T10:36:53Z

Pull request #31692 was updated. @SiewYan, @mkirsano, @jordan-martins, @chayanit, @wajidalikhan, @kpedro88, @cmsbuild, @GurpreetSinghChahal, @agrohsje, @alberto-sanchez can you please check and sign again.

chayanit · 2020-10-07T11:31:21Z

please test

cmsbuild · 2020-10-07T11:31:45Z

The tests are being triggered in jenkins.

CMSSW_11_2_X_2020-10-06-2300/slc7_amd64_gcc820: https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/9771/console Started: 2020/10/07 13:32

cmsbuild · 2020-10-07T12:43:04Z

+1
Tested at: 011392c
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-bc84d1/9771/summary.html
CMSSW: CMSSW_11_2_X_2020-10-06-2300
SCRAM_ARCH: slc7_amd64_gcc820

cmsbuild · 2020-10-07T12:43:07Z

Comparison job queued.

chayanit · 2020-10-08T06:42:44Z

To @agrohsje, would you agree the replacement of "DisplacedSUSY_stopToB_M_300_1000mm_13" gen fragment with "DisplacedSUSY_stopToB_M_800_500mm_13"? or do you like to have a new fragment instead?

agrohsje · 2020-10-08T12:43:49Z

Hi @chayanit . No objections from my side.

+generators

chayanit · 2020-10-08T12:49:43Z

+1

seems the bot can't recognise your signature above @agrohsje

agrohsje · 2020-10-08T12:52:26Z

+generators

kpedro88 · 2020-10-08T15:07:17Z

+upgrade

cmsbuild · 2020-10-08T15:07:44Z

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @silviodonato, @dpiparo, @qliphy (and backports should be raised in the release meeting by the corresponding L2)

qliphy · 2020-10-09T00:36:20Z

+1

smuzaffar · 2020-10-12T18:47:47Z

@jalimena , looks like the new data set /RelValDisplacedSUSY_stopToBottom_M_800_500mm_13/CMSSW_10_6_0-106X_mcRun2_asymptotic_v3-v1/GEN-SIM is not yet available ( DAS does not know about it ). This is causing workflow 1355 to fail , do we have this data set produced?

[a]
dasgoclient --limit 0 --query 'file dataset=/RelValDisplacedSUSY_stopToBottom_M_800_500mm_13/CMSSW_10_6_0-106X_mcRun2_asymptotic_v3-v1/GEN-SIM'

jalimena · 2020-10-12T19:37:55Z

hi @smuzaffar , no, I don't have this dataset produced yet. Should I generate it privately? Have I made a mistake in the PR?

chayanit · 2020-10-13T13:35:22Z

Configuration/PyReleaseValidation/python/relval_steps.py

@@ -3220,7 +3219,7 @@ def gen2021HiMix(fragment,howMuch):
 defaultDataSets.update(puDataSets)

 # sometimes v1 won't be used - override it here - the dictionary key is gen fragment + '_' + geometry
-versionOverrides={'BuMixing_BMuonFilter_forSTEAM_13TeV_TuneCUETP8M1_2017':'2','HSCPstop_M_200_TuneCUETP8M1_13TeV_pythia8_2017':'2','RSGravitonToGammaGamma_kMpl01_M_3000_TuneCUETP8M1_13TeV_pythia8_2017':'2','WprimeToENu_M-2000_TuneCUETP8M1_13TeV-pythia8_2017':'2','DisplacedSUSY_stopToBottom_M_300_1000mm_TuneCUETP8M1_13TeV_pythia8_2017':'2','TenE_E_0_200_pythia8_2017':'2','TenE_E_0_200_pythia8_2017PU':'2', 'TenTau_E_15_500_pythia8_2018':'2','PhotonJet_Pt_10_13TeV_TuneCUETP8M1_2018':'2','Wjet_Pt_80_120_13TeV_TuneCUETP8M1_2018':'2'}
+versionOverrides={'BuMixing_BMuonFilter_forSTEAM_13TeV_TuneCUETP8M1_2017':'2','HSCPstop_M_200_TuneCUETP8M1_13TeV_pythia8_2017':'2','RSGravitonToGammaGamma_kMpl01_M_3000_TuneCUETP8M1_13TeV_pythia8_2017':'2','WprimeToENu_M-2000_TuneCUETP8M1_13TeV-pythia8_2017':'2','DisplacedSUSY_stopToBottom_M_800_500mm_TuneCP5_13TeV_pythia8_2017':'2','TenE_E_0_200_pythia8_2017':'2','TenE_E_0_200_pythia8_2017PU':'2', 'TenTau_E_15_500_pythia8_2018':'2','PhotonJet_Pt_10_13TeV_TuneCUETP8M1_2018':'2','Wjet_Pt_80_120_13TeV_TuneCUETP8M1_2018':'2'}


This line is not necessary to change I think @jalimena

chayanit · 2020-10-13T13:36:48Z

Configuration/PyReleaseValidation/python/relval_steps.py

@@ -602,8 +602,7 @@ def gen2021hiprod(fragment,howMuch):
 steps['HSCPstop_M_200_13']=gen2015('HSCPstop_M_200_TuneCUETP8M1_13TeV_pythia8_cff',Kby(9,100))
 steps['RSGravitonToGaGa_13']=gen2015('RSGravitonToGammaGamma_kMpl01_M_3000_TuneCUETP8M1_13TeV_pythia8_cfi',Kby(9,100))
 steps['WpToENu_M-2000_13']=gen2015('WprimeToENu_M-2000_TuneCUETP8M1_13TeV-pythia8_cff',Kby(9,100))
-steps['DisplacedSUSY_stopToBottom_M_300_1000mm_13']=gen2015('DisplacedSUSY_stopToBottom_M_300_1000mm_TuneCUETP8M1_13TeV_pythia8_cff',Kby(9,100))
-
+steps['DisplacedSUSY_stopToBottom_M_800_500mm_13']=gen2015('DisplacedSUSY_stopToBottom_M_800_500mm_TuneCP5_13TeV_pythia8_cff',Kby(9,100))


I propose we keep M300 and 1000mm for 2015 (&2016) conditions here

chayanit · 2020-10-13T13:37:00Z

Configuration/PyReleaseValidation/python/relval_steps.py

@@ -780,7 +779,7 @@ def identitySim(wf):
 steps['HSCPstop_M_200_13INPUT']={'INPUT':InputInfo(dataSet='/RelValHSCPstop_M_200_13/%s/GEN-SIM'%(baseDataSetRelease[3],),location='STD')}
 steps['RSGravitonToGaGa_13INPUT']={'INPUT':InputInfo(dataSet='/RelValRSGravitonToGaGa_13/%s/GEN-SIM'%(baseDataSetRelease[3],),location='STD')}
 steps['WpToENu_M-2000_13INPUT']={'INPUT':InputInfo(dataSet='/RelValWpToENu_M-2000_13/%s/GEN-SIM'%(baseDataSetRelease[3],),location='STD')}
-steps['DisplacedSUSY_stopToBottom_M_300_1000mm_13INPUT']={'INPUT':InputInfo(dataSet='/RelValDisplacedSUSY_stopToBottom_M_300_1000mm_13/%s/GEN-SIM'%(baseDataSetRelease[3],),location='STD')}
+steps['DisplacedSUSY_stopToBottom_M_800_500mm_13INPUT']={'INPUT':InputInfo(dataSet='/RelValDisplacedSUSY_stopToBottom_M_800_500mm_13/%s/GEN-SIM'%(baseDataSetRelease[3],),location='STD')}


let's keep M300 and 1000mm for 2015 (&2016) conditions here

chayanit · 2020-10-13T13:38:15Z

Configuration/PyReleaseValidation/python/upgradeWorkflowComponents.py

@@ -1144,7 +1144,7 @@ def __init__(self, howMuch, dataset):
    ('HSCPstop_M_200_TuneCUETP8M1_13TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'HSCPstop_M_200_13')),
    ('RSGravitonToGammaGamma_kMpl01_M_3000_TuneCUETP8M1_13TeV_pythia8_cfi', UpgradeFragment(Kby(9,50),'RSGravitonToGaGa_13')),
    ('WprimeToENu_M-2000_TuneCUETP8M1_13TeV-pythia8_cff', UpgradeFragment(Kby(9,50),'WpToENu_M-2000_13')),
-    ('DisplacedSUSY_stopToBottom_M_300_1000mm_TuneCUETP8M1_13TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'DisplacedSUSY_stopToB_M_300_1000mm_13')),
+    ('DisplacedSUSY_stopToBottom_M_800_500mm_TuneCP5_13TeV_pythia8_cff', UpgradeFragment(Kby(9,50),'DisplacedSUSY_stopToB_M_800_500mm_13')),


I propose to update only upgrade workflows and leave standard workflow as it is

chayanit · 2020-10-13T13:39:17Z

Configuration/PyReleaseValidation/python/relval_standard.py

@@ -660,7 +660,7 @@
 workflows[1352] = ['', ['HSCPstop_M_200_13','DIGIUP15','RECOUP15','HARVESTUP15']]
 workflows[1353] = ['', ['RSGravitonToGaGa_13','DIGIUP15','RECOUP15','HARVESTUP15']]
 workflows[1354] = ['', ['WpToENu_M-2000_13','DIGIUP15','RECOUP15','HARVESTUP15']]
-workflows[1355] = ['', ['DisplacedSUSY_stopToBottom_M_300_1000mm_13','DIGIUP15','RECOUP15','HARVESTUP15']]
+workflows[1355] = ['', ['DisplacedSUSY_stopToBottom_M_800_500mm_13','DIGIUP15','RECOUP15','HARVESTUP15']]


I propose we update only upgrade workflows and leave standard workflow as the old sample since we don't produce this 1355 anymore

chayanit · 2020-10-13T13:43:07Z

Hi @jalimena , may I propose we update only "upgrade" workflow in upgradeWorkflowComponents.py and leave everything in relval_standard as it is? We actually do not run 2015 workflow anymore. So then we may need to have the old fragment back in Configuration/Genproduction package? Sorry for this mess!

remove DisplacedSUSY_stopToB_M_300_1000mm_13 and add DisplacedSUSY_st…

9503933

…opToB_M_800_500mm_13/14 gen fragments and in WFs

cmsbuild added this to the CMSSW_11_2_X milestone Oct 7, 2020

cmsbuild added code-checks-pending comparison-pending generators-pending orp-pending pdmv-pending pending-signatures tests-pending upgrade-pending labels Oct 7, 2020

cmsbuild added code-checks-approved and removed code-checks-pending labels Oct 7, 2020

chayanit suggested changes Oct 7, 2020

View reviewed changes

moved new WF to the bottom, in order to not ruin the order of existin…

011392c

…g workflow IDs

cmsbuild added code-checks-pending and removed code-checks-approved labels Oct 7, 2020

cmsbuild added code-checks-approved and removed code-checks-pending labels Oct 7, 2020

cmsbuild added tests-started and removed tests-pending labels Oct 7, 2020

This was referenced Oct 7, 2020

Nano gen dqm #31591

Merged

Sub-event aware genJet and parton matching for HI miniAOD #31698

Merged

Bad PF candidate filter for 2018 PbPb data #31668

Merged

cmsbuild mentioned this pull request Oct 8, 2020

MTD geometry: add scenarios D72 (ETL v4) and D73 (ETL v5), remove D53 #31710

Merged

cmsbuild added pdmv-approved and removed pdmv-pending labels Oct 8, 2020

cmsbuild added generators-approved and removed generators-pending labels Oct 8, 2020

cmsbuild added fully-signed upgrade-approved and removed pending-signatures upgrade-pending labels Oct 8, 2020

cmsbuild mentioned this pull request Oct 8, 2020

Integrating VectorHits reconstruction for the Phase2 OT - reprise #31314

Merged

cmsbuild added orp-approved and removed orp-pending labels Oct 9, 2020

cmsbuild merged commit a03f63d into cms-sw:master Oct 9, 2020

chayanit reviewed Oct 13, 2020

View reviewed changes

jalimena mentioned this pull request Oct 13, 2020

fix for displaced susy relval production (TRK and EXO validation of displaced tracks) #31772

Merged

jalimena deleted the DispSUSYRelVals branch November 11, 2020 15:28

CeliaFernandez mentioned this pull request Feb 16, 2023

Fix double counting in HepMC to G4 handoff #39427

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

update of "DisplacedSUSY" relvals for TRK and EXO validation of displaced tracks, for Run 3 and Phase 2 #31692

update of "DisplacedSUSY" relvals for TRK and EXO validation of displaced tracks, for Run 3 and Phase 2 #31692

jalimena commented Oct 7, 2020 •

edited

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

chayanit Oct 7, 2020

jalimena Oct 7, 2020

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

chayanit commented Oct 7, 2020

cmsbuild commented Oct 7, 2020 •

edited

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

chayanit commented Oct 8, 2020

agrohsje commented Oct 8, 2020 •

edited

chayanit commented Oct 8, 2020

agrohsje commented Oct 8, 2020

kpedro88 commented Oct 8, 2020

cmsbuild commented Oct 8, 2020

qliphy commented Oct 9, 2020

smuzaffar commented Oct 12, 2020

jalimena commented Oct 12, 2020

chayanit Oct 13, 2020

chayanit Oct 13, 2020

chayanit Oct 13, 2020

chayanit Oct 13, 2020

chayanit Oct 13, 2020

chayanit commented Oct 13, 2020 •

edited

update of "DisplacedSUSY" relvals for TRK and EXO validation of displaced tracks, for Run 3 and Phase 2 #31692

update of "DisplacedSUSY" relvals for TRK and EXO validation of displaced tracks, for Run 3 and Phase 2 #31692

Conversation

jalimena commented Oct 7, 2020 • edited

PR description:

PR validation:

if this PR is a backport please specify the original PR and why you need to backport that PR:

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

chayanit Oct 7, 2020

Choose a reason for hiding this comment

jalimena Oct 7, 2020

Choose a reason for hiding this comment

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

chayanit commented Oct 7, 2020

cmsbuild commented Oct 7, 2020 • edited

cmsbuild commented Oct 7, 2020

cmsbuild commented Oct 7, 2020

chayanit commented Oct 8, 2020

agrohsje commented Oct 8, 2020 • edited

chayanit commented Oct 8, 2020

agrohsje commented Oct 8, 2020

kpedro88 commented Oct 8, 2020

cmsbuild commented Oct 8, 2020

qliphy commented Oct 9, 2020

smuzaffar commented Oct 12, 2020

jalimena commented Oct 12, 2020

chayanit Oct 13, 2020

Choose a reason for hiding this comment

chayanit Oct 13, 2020

Choose a reason for hiding this comment

chayanit Oct 13, 2020

Choose a reason for hiding this comment

chayanit Oct 13, 2020

Choose a reason for hiding this comment

chayanit Oct 13, 2020

Choose a reason for hiding this comment

chayanit commented Oct 13, 2020 • edited

jalimena commented Oct 7, 2020 •

edited

cmsbuild commented Oct 7, 2020 •

edited

agrohsje commented Oct 8, 2020 •

edited

chayanit commented Oct 13, 2020 •

edited