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

Update to the alphaT HLT DQM 74X #6876

Merged
merged 5 commits into from Dec 19, 2014

Conversation

aelwood
Copy link
Contributor

@aelwood aelwood commented Dec 11, 2014

Updated the alphaT DQM module to make plots for PF and Calo Jets, in line with the inclusion of PF Jets with a Calo prefilter for the alphaT HLT paths. ( /users/aelwood/alphaT/alphaTRun2PF/V6 ).

Associated with PR for 73X:
#6874

violatingcp and others added 5 commits December 10, 2014 09:54
fixed chi2 constraint on pedestal and time
…SSW_7_4_X

Conflicts:
	RecoLocalCalo/HcalRecAlgos/src/PulseShapeFitOOTPileupCorrection.cc
…line with the inclusion of PF Jets with a Calo prefilter for the alphaT HLT paths
@cmsbuild
Copy link
Contributor

A new Pull Request was created by @aelwood for CMSSW_7_4_X.

Update to the alphaT HLT DQM 74X

It involves the following packages:

HLTriggerOffline/SUSYBSM

@cmsbuild, @danduggan, @nclopezo, @deguio can you please review it and eventually sign? Thanks.
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.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

@deguio
Copy link
Contributor

deguio commented Dec 19, 2014

+1

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_4_X IBs unless changes (tests are also fine). This pull request will be automatically merged.

cmsbuild added a commit that referenced this pull request Dec 19, 2014
Update to the alphaT HLT DQM 74X
@cmsbuild cmsbuild merged commit 27feba7 into cms-sw:CMSSW_7_4_X Dec 19, 2014
TriggerPath = cms.string('HLT_HT200_DiJet90_AlphaT0p57_v'),
TriggerPathAuxiliaryForHadronic = cms.string('HLT_IsoTkMu24_eta2p1_IterTrk02_v'),
TriggerFilter = cms.InputTag('hltHT200CaloAlphaT0p57', '', 'HLT'), #the last filter in the path
TriggerPath = cms.string('HLT_PFHT200_PFDiJet90_PFAlphaT0p57_v'),
Copy link
Contributor

Choose a reason for hiding this comment

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

The name of the path is HLT_PFHT200_DiPFJet90_PFAlphaT0p57 (and the same for the other three similar paths dealth with in this pull request), see JIRA ticket CMSHLT-207
Please provide a fix with the correct HLT path name: only at that point those paths will be allowed in the HLT GRun menu.

@deguio
Copy link
Contributor

deguio commented Jan 26, 2015

thanks @perrotta for noticing this.
the fix will hopefully reduce the verbosity of this module as well.

@aelwood could you confirm that you can take care of this?
thanks in advance,
F.

@perrotta
Copy link
Contributor

@deguio : please, find a fix in #7385
If it is fine, TSG will readmit those paths in the Run2 HLT menu

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

6 participants