Skip to content

Needed fixes to run with photos#50915

Open
alebihan wants to merge 2 commits into
cms-sw:masterfrom
alebihan:embeddingfilter_CMSSW_17_0_X
Open

Needed fixes to run with photos#50915
alebihan wants to merge 2 commits into
cms-sw:masterfrom
alebihan:embeddingfilter_CMSSW_17_0_X

Conversation

@alebihan
Copy link
Copy Markdown

PR description:

Two protections added to avoid crashed when using photos ("null vertices of intermediate 746 particles")

PR validation:

Tested on 10k events with cmsDriver.py Configuration/GenProduction/python/SMP-RunIII2024Summer24wmLHEGS-00182-fragment.py --eventcontent RAWSIM,LHE --customise Configuration/DataProcessing/Utils.addMonitoring --datatier GEN-SIM,LHE --conditions 140X_mcRun3_2024_realistic_v26 --beamspot DBrealistic...

If this PR is a backport please specify the original PR and why you need to backport that PR. If this PR will be backported please specify to which release cycle the backport is meant for:

This PR is made to the master branch to be backported to 140X releases to produce a 2024 sample for the dedicated DY production needed for tau polarisation studies

@cmsbuild
Copy link
Copy Markdown
Contributor

cmsbuild commented May 11, 2026

cms-bot internal usage

@cmsbuild
Copy link
Copy Markdown
Contributor

-code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50915/49294

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

@cmsbuild
Copy link
Copy Markdown
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-50915/49296

@cmsbuild
Copy link
Copy Markdown
Contributor

A new Pull Request was created by @alebihan for master.

It involves the following packages:

  • GeneratorInterface/Core (generators)

@cmsbuild, @lviliani, @mkirsano, @sensrcn, @theofil can you please review it and eventually sign? Thanks.
@alberto-sanchez, @mkirsano 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

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.

2 participants