Changes to Info class so that weights do now grow with every re-init

[smrenna]

Chris Jones convinced me to do a pull request instead ;).;

[cmsbuild]

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

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

[mrodozov]

please test

[cmsbuild]

The tests are being triggered in jenkins.
https://cmssdt.cern.ch/jenkins/job/ib-run-pr-tests/2441/console Started: 2019/09/10 09:47

[cmsbuild]

+1
Tested at: f519558
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6b4749/2441/summary.html

[cmsbuild]

Comparison job queued.

[cmsbuild]

Comparison is ready
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6b4749/2441/summary.html

Comparison Summary:

• No significant changes to the logs found
• Reco comparison results: 4 differences found in the comparisons
• DQMHistoTests: Total files compared: 34
• DQMHistoTests: Total histograms compared: 2957224
• DQMHistoTests: Total failures: 2
• DQMHistoTests: Total nulls: 0
• DQMHistoTests: Total successes: 2956881
• DQMHistoTests: Total skipped: 341
• DQMHistoTests: Total Missing objects: 0
• DQMHistoSizes: Histogram memory added: 0.0 KiB( 33 files compared)
• Checked 145 log files, 15 edm output root files, 34 DQM output files

[fabiocos]

@smrenna do you consider this as a fix to be integrated in future pythia8 releases?

[smrenna]

Yes, I did a merge request on pythia82 yesterday. I will provide more information when/if it is available. We are hoping that our next major release is pythia83, and I also made a merge request there. Steve From: Fabio Cossutti <[email protected]> Reply-To: cms-externals/pythia8 <[email protected]> Date: Tuesday, September 10, 2019 at 9:23 AM To: cms-externals/pythia8 <[email protected]> Cc: Stephen Mrenna <[email protected]>, Mention <[email protected]> Subject: Re: [cms-externals/pythia8] Changes to Info class so that weights do now grow with every re-init (#14) @smrenna<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_smrenna&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=xuVZsb_gHFnIWZESUxFvnIfKjZBlF2OwYeYRPAV_EiI&s=GIuTPc-4b59k7-eg2NW-d8nVTjvQdwDt2xo2homEFZU&e=> do you consider this as a fix to be integrated in future pythia8 releases? — 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_14-3Femail-5Fsource-3Dnotifications-26email-5Ftoken-3DABLSEJF3JR6HES46LTK2DNDQI6UWDA5CNFSM4IU7H5QKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD6LIVFI-23issuecomment-2D529959573&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=xuVZsb_gHFnIWZESUxFvnIfKjZBlF2OwYeYRPAV_EiI&s=BWVB3ZNEIvcBuqphxgtFf0kY9KmWbvW2aCur-bGrG1s&e=>, or mute the thread<https://urldefense.proofpoint.com/v2/url?u=https-3A__github.com_notifications_unsubscribe-2Dauth_ABLSEJAYVVMS7IVRRTQO74DQI6UWDANCNFSM4IU7H5QA&d=DwMCaQ&c=gRgGjJ3BkIsb5y6s49QqsA&r=P6TR6eMSzDEOLYeqW0ECsQ&m=xuVZsb_gHFnIWZESUxFvnIfKjZBlF2OwYeYRPAV_EiI&s=P8ISuapbqkEtVyN_-bBx4r8-P0kIAenksiQfOFdjp4Y&e=>.

[fabiocos]

+1

[smuzaffar]

+externals

[cmsbuild]

This pull request is fully signed and it will be integrated in one of the next cms/240 IBs (tests are also fine). This pull request will be automatically merged.