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

Add in bunch spacing producer to PAT/rePAT steps. #3

Open
wants to merge 1 commit into
base: centralBSProd
Choose a base branch
from

Conversation

lgray
Copy link

@lgray lgray commented Sep 29, 2015

@davidlange6 changes as stated in cms-sw#11544

@@ -284,6 +284,9 @@ def miniAOD_customizeCommon(process):
## Force a re-run of the tau id during MiniAOD production stage
process.load('RecoTauTag.Configuration.RecoPFTauTag_cff')

#bunch spacing producer
process.load('RecoLuminosity.LumiProducer.bunchSpacingProducer_cfi')
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shouldn't this go into a lower level pat sequence?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

MiniAOD is run only in unscheduled mode, there are no sequences.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

On Sep 29, 2015, at 1:56 PM, Lindsey Gray [email protected] wrote:

In PhysicsTools/PatAlgos/python/slimming/miniAOD_tools.py:

@@ -284,6 +284,9 @@ def miniAOD_customizeCommon(process):
## Force a re-run of the tau id during MiniAOD production stage
process.load('RecoTauTag.Configuration.RecoPFTauTag_cff')

  • #bunch spacing producer
  • process.load('RecoLuminosity.LumiProducer.bunchSpacingProducer_cfi')

MiniAOD is run in unscheduled mode, there are no sequences.

right - but this is a fairly high level config (and miniAOD specific).

maybe better in Configuration/StandardSequences/PAT_cff

but still thinking..


Reply to this email directly or view it on GitHub.

davidlange6 pushed a commit that referenced this pull request Jul 17, 2017
davidlange6 pushed a commit that referenced this pull request Aug 24, 2017
davidlange6 pushed a commit that referenced this pull request Aug 13, 2018
davidlange6 pushed a commit that referenced this pull request Dec 20, 2018
davidlange6 pushed a commit that referenced this pull request Dec 20, 2018
Use cloning in modules, Add back DeepDoubleBTagInfo.h in the meantime so PR can be tested
davidlange6 pushed a commit that referenced this pull request Apr 30, 2020
davidlange6 pushed a commit that referenced this pull request Mar 11, 2021
davidlange6 pushed a commit that referenced this pull request Jun 22, 2021
pull from cmssw/master
davidlange6 pushed a commit that referenced this pull request Jul 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants