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

Boosted jet cleaning #4809

Merged
merged 8 commits into from Jul 29, 2014
Merged

Conversation

isando3
Copy link
Contributor

@isando3 isando3 commented Jul 28, 2014

New plugin that was designed to do a lepton-jet cleaning algorithm that is very important for boosted topologies where the lepton can potentially merge into the jet. If the cleaning is not updated many important events would be thrown away due to the current inefficient jet cleaning.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @isando3 for CMSSW_7_2_X.

Boosted jet cleaning

It involves the following packages:

HLTrigger/JetMET

@perrotta, @cmsbuild, @nclopezo, @Martin-Grunewald, @fwyzard, @Degano 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

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

@Martin-Grunewald
Copy link
Contributor

+1

Presumably this is needed in 71X, for HLT development, as well, so you
need to make a (backport) PR for 71X.

@cmsbuild
Copy link
Contributor

This pull request is fully signed and it will be integrated in one of the next CMSSW_7_2_X IBs unless changes or unless it breaks tests.

@cmsbuild
Copy link
Contributor

@cmsbuild
Copy link
Contributor

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

nclopezo added a commit that referenced this pull request Jul 29, 2014
HLTrigger/JetMET -- Boosted jet cleaning
@nclopezo nclopezo merged commit c93c97a into cms-sw:CMSSW_7_2_X Jul 29, 2014
@cmsbuild
Copy link
Contributor

@isando3
Copy link
Contributor Author

isando3 commented Jul 29, 2014

Hi Martin,

If I want this plugin to be used also in ConfDB for HLT development I need
to do the same PR in 713?

Thanks.

On Tue, Jul 29, 2014 at 2:31 AM, Martin Grunewald notifications@github.com
wrote:

+1

Presumably this is needed in 71X, for HLT development, as well, so you
need to make a (backport) PR for 71X.


Reply to this email directly or view it on GitHub
#4809 (comment).

Best Regards,


Daniel Sandoval
University of Illinois at Chicago
Physics Department *
*Office: SES 2156, Lab: SES 2259

Office Phone: (+1) (312)-996-5648
Lab Phone: (+1) (312)-413-2108
Mobile Phone: (+1) (312)-593-5207
email: isando3@uic.edu isando3@uic.edu

  •        isandova@cern.ch isandova@cern.ch*
        *dansand@fnal.gov dansand@fnal.gov *
    

@Martin-Grunewald
Copy link
Contributor

Yes: PRs are made for CMSSW release series, so 71X and 72X etc.
You just make a PR for 71X which will eventually be integrated in
the next 71X release, and then we parse that release it into ConfDB.

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