Phase2 Single_Tau_Trigger Path Added

[Chihwan-An]

Added Single Tau Trigger path for Phase2 , worked with Tau HLT Phase2 development group.

It was tested in miniAOD in CMSSW_15_1_0_pre4 and NanoAOD in CMSSW_16_0_0_pre2 , with Z'to tautau samples.

CMSSW_16_0_0_pre2 version was used to check trigger efficiency , which CMSSW_15_1_0_pre4 has not full resource to get 'genvistau' object .

Trigger Efficiency had 48.7% .

[cmsbuild]

A new Pull Request was created by @Chihwan-An for CMSSW_15_1_X.

It involves the following packages:

• HLTrigger/Configuration (hlt)

@Martin-Grunewald, @cmsbuild, @mmusich can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @SohamBhattacharya, @VourMa, @missirol, @mmusich, @rovere this is something you requested to watch as well.
@ftenchini, @mandrenguyen, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

[cmsbuild]

cms-bot internal usage

[Martin-Grunewald]

You'd need to provide a PR to 16_0 first, then denote this PR as its backport.
Also, please add in the description the link to the TSG presentation where this path was discussed.

[mmusich]

@Chihwan-An
Indeed as Martin said, you need to first produce a PR for master (currently CMSSW_16_0_X). Also why do you need this in CMSSW_15_1_X?
In my opinion it's not needed (so please close this one).
Also here are some comments. Please apply them before opening the PR in master.

[mmusich]

a newline is missing here.

[mmusich]

why is MaxEta = 2.1, but MinEta = -1.0?
Why the asymmetric cut?

[Chihwan-An]

As I Know , MinEta (-1.0) means default value of minmum value.

[sakura-ngt]

As I Know , MinEta (-1.0) means default value of minmum value.

Checking the code, this doesn't seem to be the case.

cmssw/HLTrigger/HLTfilters/plugins/plugins.cc

Line 60 in ff1c182

typedef HLTSinglet<PFTau> HLT1PFTau;

and

cmssw/HLTrigger/HLTfilters/plugins/HLTSinglet.cc

Lines 145 to 149 in ff1c182

	for (; i != objects->end(); i++) {
	if ((i->energy() >= min_E_) && (i->pt() >= min_Pt_) && (i->mass() >= min_Mass_) &&
	((max_Mass_ < 0.0) || (i->mass() <= max_Mass_)) && ((min_Eta_ < 0.0) || (std::abs(i->eta()) >= min_Eta_)) &&
	((max_Eta_ < 0.0) || (std::abs(i->eta()) <= max_Eta_))) {
	n++;

which means that:

• if MaxEta > 0, then accept for |η|< MaxEta
• if MinEta > 0, then accept for |η|> MinEta

the choice of parameter names is not great. IMHO WithinEta and OutsideOfEta might be better choices. Something to consider when we're free to develop outside of the constraint to support Run3.

[mmusich]

this whole file is useless at the moment, we don't have prescales in the phase-2 menu.
Please remove.

[mmusich]

Suggested change

please remove empty lines.

[Chihwan-An]

@Chihwan-An Indeed as Martin said, you need to first produce a PR for master (currently CMSSW_16_0_X). Also why do you need this in CMSSW_15_1_X? In my opinion it's not needed (so please close this one). Also here are some comments. Please apply them before opening the PR in master.

Thanks for nice comments .
CMSSW_15_1_X was used due to scram list , there was no CMSSW_16_0_X .
I choosed he most recent avaliable version which was CMSSW_15_1_X .
Sorry for my mistake , I thougt I should make PR in version I used.

[Chihwan-An]

You'd need to provide a PR to 16_0 first, then denote this PR as its backport. Also, please add in the description the link to the TSG presentation where this path was discussed.

I think it is better to PR to 16_0_X , I'll make new PR for that version. Thanks.