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

Backport of HLTJetTagWithMatching in 73X (#6985) #7087

Merged
merged 2 commits into from Jan 13, 2015

Conversation

silviodonato
Copy link
Contributor

Backport of #6985


"
I'm proposing to add the HLTJetTagWithMatching module in CMSSW.
This module is very similar to HLTJetTag.

HLTJetTag is a filter that pass events matching some requirements on the jetTag (eg. CSV>0.7).
It uses a jet InputTag only to fill the TriggerFilterObjectWithRefs.

HLTJetTagWithMatching is a filter that loop on a jet collection, and each jet is matched with the jetTag.
In this way it use the jet information from the jet collection, and the b-tag information by the jetTag collection.

With the new module, it will be possible, for instance, to have filter like "at least one jet with pT>50 GeV and CSV>0.7". In fact, you can produces a subset of jets and give it as InputTag to this module.

Silvio.
"

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 8, 2015

A new Pull Request was created by @silviodonato for CMSSW_7_3_X.

Backport in 73X of #6985

It involves the following packages:

HLTrigger/btau

@Martin-Grunewald, @perrotta, @cmsbuild, @nclopezo, @fwyzard can you please review it and eventually sign? Thanks.
@Martin-Grunewald, @ferencek, @acaudron, @pvmulder, @imarches 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' in the first line of a comment.
@nclopezo you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@perrotta
Copy link
Contributor

perrotta commented Jan 8, 2015

wrt #6985, you do not include here the definition of the new module in HLTrigger/btau/src/modules.cc

@silviodonato
Copy link
Contributor Author

Thanks! I've just added it.

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 9, 2015

Pull request #7087 was updated. @Martin-Grunewald, @perrotta, @cmsbuild, @nclopezo, @fwyzard can you please check and sign again.

@perrotta
Copy link
Contributor

perrotta commented Jan 9, 2015

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 9, 2015

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes or unless it breaks tests. This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

@cmsbuild
Copy link
Contributor

cmsbuild commented Jan 9, 2015

@Martin-Grunewald
Copy link
Contributor

+1
(Wrong PR given -1)

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_3_X IBs unless changes (tests are also fine). This pull request requires discussion in the ORP meeting before it's merged. @davidlange6, @nclopezo, @ktf, @smuzaffar

@davidlange6
Copy link
Contributor

+1

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

5 participants