msafety set to 1e-4

[smrenna]

Request from Sam Bein for lower threshold on MSAFETY in ResonanceDecays (only) to allow for pMSSM decays with really small mass splittings.

[cmsbuild]

A new Pull Request was created by @smrenna (Stephen Mrenna) for branch cms/230.

@cmsbuild, @smuzaffar, @mrodozov can you please review it and eventually sign? Thanks.
cms-bot commands are listed here

[qliphy]

please test for CMSSW_10_2_X

[cmsbuild]

The tests are being triggered in jenkins.
Test Parameters:

• RELEASE_FORMAT = CMSSW_10_2_X
• CMSSW_10_2_X_2020-11-23-2300/slc6_amd64_gcc700: https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/10969/console Started: 2020/11/24 04:11

[qliphy]

@smrenna Should this also be included in other branches? e.g.
https://github.com/cms-externals/pythia8/blob/cms/240/src/ResonanceDecays.cc#L30

[smrenna]

I don’t think so. Alexander suggested to do this just for cms/230 From: Qiang Li <notifications@github.com> Reply-To: cms-externals/pythia8 <reply@reply.github.com> Date: Monday, November 23, 2020 at 6:27 PM To: cms-externals/pythia8 <pythia8@noreply.github.com> Cc: Stephen Mrenna <mrenna@fnal.gov>, Mention <mention@noreply.github.com> Subject: Re: [cms-externals/pythia8] msafety set to 1e-4 (#26) @smrenna<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_smrenna&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=O8Qohm8J4fcrxi_zGq5z0r-ux1WSO797kKotA5M66zg&s=yBRYO5K7M3EsC5zxcj8FF3ZWb5FeNTUZhLuOQ0aRH24&e=> Should this also be included in other branches? e.g. https://github.com/cms-externals/pythia8/blob/cms/240/src/ResonanceDecays.cc#L30<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_cms-2Dexternals_pythia8_blob_cms_240_src_ResonanceDecays.cc-23L30&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=O8Qohm8J4fcrxi_zGq5z0r-ux1WSO797kKotA5M66zg&s=Op5xU5ZiaUbzZ5A2z5NjSJkhP9AzqDiArJ0QRNpxFwQ&e=> — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_cms-2Dexternals_pythia8_pull_26-23issuecomment-2D732504627&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=O8Qohm8J4fcrxi_zGq5z0r-ux1WSO797kKotA5M66zg&s=40yhXTE7lJQigblIMuGWcaLci5A3JZo655ZlVwosYvE&e=>, or unsubscribe<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABLSEJGQZ73P4MHOHF3YRZLSRL4W7ANCNFSM4UAEUEHQ&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=O8Qohm8J4fcrxi_zGq5z0r-ux1WSO797kKotA5M66zg&s=nwSDdMUKZnv9dNU_yYIBv6HCRdV6h5InULoM8A84HSY&e=>.

[cmsbuild]

+1
Tested at: 57ae2d4
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3f4ad6/10969/summary.html
CMSSW: CMSSW_10_2_X_2020-11-23-2300
SCRAM_ARCH: slc6_amd64_gcc700

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-3f4ad6/10969/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 0 differences found in the comparisons
• DQMHistoTests: Total files compared: 31
• DQMHistoTests: Total histograms compared: 3007546
• DQMHistoTests: Total failures: 2
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 3007354
• DQMHistoTests: Total skipped: 190
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 30 files compared)
• Checked 129 log files, 25 edm output root files, 31 DQM output files

[agrohsje]

Hi @qliphy . My main motivation was to not move too far away from the P8 default. As this is needed for SUSY and they can live with 10_2 and 9_4 I would prefer just updating P8.230. What do you think?

[qliphy]

@agrohsje ok, that sounds good. This PR is similar as #20 , but further reduce msafety.

[agrohsje]

Yes, should be identical to #20 but with lower values.

[agrohsje]

assign generators

[cmsbuild]

New categories assigned: generators

@alberto-sanchez,@SiewYan,@GurpreetSinghChahal,@mkirsano,@agrohsje you have been requested to review this Pull request/Issue and eventually sign? Thanks

[agrohsje]

+1

[mrodozov]

so this goes in 10_2 ?

[agrohsje]

10_2 and 9_4.

[silviodonato]

cmsdist PRs cms-sw/cmsdist#6469 and cms-sw/cmsdist#6470