-
Notifications
You must be signed in to change notification settings - Fork 4.6k
auto detect first vertex with Higgs in children list #49644
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
base: master
Are you sure you want to change the base?
Conversation
|
cms-bot internal usage |
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49644/47181 ERROR: Unable to merge PR. See log https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49644/47181/cms-checkout-topic.log |
f78beb8 to
6844d63
Compare
|
-code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49644/47184 Code check has found code style and quality issues which could be resolved by applying following patch(s)
|
6844d63 to
e2f24e0
Compare
|
+code-checks Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-49644/47185 |
|
A new Pull Request was created by @yihui-lai for master. It involves the following packages:
@cmsbuild, @lviliani, @mkirsano, @sensrcn, @theofil can you please review it and eventually sign? Thanks. cms-bot commands are listed here |
|
Milestone for this pull request has been moved to CMSSW_16_1_X. Please open a backport if it should also go in to CMSSW_16_0_X. |
PR description:
The current HTXS production-mode identification relies on inspecting GenEvent::vertices()[0] to determine the Higgs boson’s production topology. However, for certain
MG5_aMC@NLO FxFxsamples—such asTTH-Hto2G_Par-M-125_TuneCP5_13p6TeV_amcatnloFXFX-pythia8—the Higgs does not appear among the children of vertices()[0].As a result, the HTXS Rivet module fails to assign a valid Higgs production mode, which propagates to a failure in HTXS categorization during MiniAOD→NanoAOD production. This PR updates the logic to automatically identify the first GenVertex that contains a Higgs boson among its outgoing particles, rather than assuming it is located at vertices()[0].
PR validation:
The updated logic was validated by running MiniAOD → NanoAOD workflows on both:
In all tested samples, the HTXS production mode and category are now correctly assigned.
@mseidel42