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

Toplhefilter #27795

Closed
wants to merge 902 commits into from
Closed

Toplhefilter #27795

wants to merge 902 commits into from

Conversation

ohindrichs
Copy link

PR description:

This is a filter acting on the LHE output. It clusters the final LHE particles using the anti-kt jet algorithm. A cut on the minimum pT > 350 GeV is selected. This filter is a loose pre-selection of the AK8-jet pT > 650 GeV filter at particle level used in the production of TT_AK8HT650* samples. The pre-filter selects at LHE level 99.8% of the events with a jet of pT > 650 GeV at particle level, while only 3% of the inclusive ttbar sample passes the filter. Therefore, the number of events that have to be showered before the filter decision is drastically reduced.

PR validation:

Tests have shown that this pre-filter reduces the the run time per event from ~1.3s to ~0.2s.

ssekmen and others added 30 commits January 9, 2019 05:45
Implement a global EDFilter "BunchCrossingFilter" to select events based
on their bunch crossing number (between 1 and 3564 inclusive).
Implement an EDFilter to select events based on their bunch crossing number (10.2.x)
…s_102X

Backport FastSim2017 PU and pmx wfs to 10_2_X
moved low pT ele sequence into highlevelreco
@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-27795/11493

Code check has found code style and quality issues which could be resolved by applying following patch(s)

@franzoni
Copy link

hello @ohindrichs
this pr seem to need rebasing...

@franzoni
Copy link

After addressing @srimanob 's comment here:

can you pls make sure that besides the 10_2 back-port
(here)
you also add a 10_6 backport ?

@cmsbuild
Copy link
Contributor

The code-checks are being triggered in jenkins.

@cmsbuild
Copy link
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-27795/11512

Code check has found code style and quality issues which could be resolved by applying following patch(s)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment