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

Prompt skims for PbPb and pp at 5 TeV #12243

Merged
merged 8 commits into from Nov 10, 2015

Conversation

mandrenguyen
Copy link
Contributor

Note that the final HLT emulation has not been added to the release yet.
However these skims will not crash, they just bail if the trigger path isn't present.

We tested the reco-level filters on a sample with the HLT menu which is about to be put into the release, following a recipe provided by DDT.

If anyone is interested, the recipe follows.

Example for PbPb:
cmsDriver.py skims -s SKIM:Photon --scenario HeavyIons --data --filein=file:/afs/cern.ch/work/m/mnguyen/public/promptSkimTestInput/hiReco_RAW2DIGI_L1Reco_RECO_forPromptSkim_ttbar_FEVTDEBUGHLT.root -n 1000 --conditions auto:com10 --python_filename=test.py --processName=SKIM --no_exec

Example for pp:
skims -s SKIM:Onia --data --filein=file:/afs/cern.ch/work/m/mnguyen/public/promptSkimTestInput/ppReco_RAW2DIGI_L1Reco_RECO_forPromptSkim_ttbar_FEVTDEBUGHLT.root -n 1000 --conditions auto:com10 --python_filename=oniaZMM.py --processName=SKIM --no_exec

The HLT re-emulation was done with a different label, so you need to add the following snippet by hand to the cfg:
from HLTrigger.Configuration.CustomConfigs import MassReplaceInputTag
process = MassReplaceInputTag(process,cms.InputTag("TriggerResults","","HLT"),cms.InputTag("TriggerResults","","TEST"))

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 3, 2015

A new Pull Request was created by @mandrenguyen for CMSSW_7_5_X.

Prompt skims for PbPb and pp at 5 TeV

It involves the following packages:

Configuration/StandardSequences
HeavyIonsAnalysis/Configuration

@cmsbuild, @vadler, @franzoni, @davidlange6, @monttj can you please review it and eventually sign? Thanks.
@ghellwig, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @Martin-Grunewald, @istaslis, @cerati, @dgulhan, @yetkinyilmaz 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' or '@cmsbuild, 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.

@davidlange6
Copy link
Contributor

please test

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 7, 2015

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 7, 2015

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 7, 2015

@davidlange6
Copy link
Contributor

@mandrenguyen - could you move these configs to Configuration/Skimming? I would prefer that the analysis packages are kept for non-production analysis rather than for production.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 8, 2015

Pull request #12243 was updated. @danduggan, @boudoul, @franzoni, @fabozzi, @cmsbuild, @srimanob, @vadler, @monttj, @davidlange6 can you please check and sign again.

@mandrenguyen
Copy link
Contributor Author

@davidlange6
I updated according to your request.

@davidlange6
Copy link
Contributor

Thanks ! (as a added benefit those organizing the skims now are on the signature list)

On Nov 8, 2015, at 2:36 PM, mandrenguyen notifications@github.com wrote:

@davidlange6
I updated according to your request.


Reply to this email directly or view it on GitHub.

@mandrenguyen
Copy link
Contributor Author

@davidlange6 How about a please test ?

@davidlange6
Copy link
Contributor

please test

(good point..)

On Nov 8, 2015, at 5:14 PM, mandrenguyen notifications@github.com wrote:

@davidlange6 How about a please test ?


Reply to this email directly or view it on GitHub.

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 8, 2015

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

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 8, 2015

@cmsbuild
Copy link
Contributor

cmsbuild commented Nov 8, 2015

@fabozzi
Copy link
Contributor

fabozzi commented Nov 10, 2015

+1

@cmsbuild
Copy link
Contributor

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

@cmsbuild
Copy link
Contributor

Pull request #12243 was updated. @cmsbuild, @vadler, @franzoni, @davidlange6, @monttj can you please check and sign again.

davidlange6 added a commit that referenced this pull request Nov 10, 2015
Prompt skims for PbPb and pp at 5 TeV
@davidlange6 davidlange6 merged commit e37b8de into cms-sw:CMSSW_7_5_X Nov 10, 2015
@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

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