-
Notifications
You must be signed in to change notification settings - Fork 184
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
[SimG4CMS-Calo] Update HFShowerLibrary_run3_v7 #8292
Conversation
A new Pull Request was created by @smuzaffar (Malik Shahzad Muzaffar) for branch IB/CMSSW_13_0_X/master. @cmsbuild, @smuzaffar, @aandvalenzuela, @iarspider can you please review it and eventually sign? Thanks. |
FYI @bsunanda , this Pr should provide the updated HF shower library |
Thanks Shahzad, Does it mean that I dont have to push the changes any more?
…________________________________
From: Malik Shahzad Muzaffar ***@***.***
Sent: 06 February 2023 12:59
To: cms-sw/cmsdist
Cc: Sunanda Banerjee; Mention
Subject: Re: [cms-sw/cmsdist] [SimG4CMS-Calo] Update HFShowerLibrary_run3_v7 (PR #8292)
FYI @bsunanda<https://github.com/bsunanda> , this Pr should provide the updated HF shower library
—
Reply to this email directly, view it on GitHub<#8292 (comment)>, or unsubscribe<https://github.com/notifications/unsubscribe-auth/ABGMZOXR5QZ2FT3TMB3CEGTWWDRR5ANCNFSM6AAAAAAUSSVXYI>.
You are receiving this because you were mentioned.Message ID: ***@***.***>
|
yes @bsunanda , this is a special data package which contains more than 100MB file and github does not allow that. For this data package we download the datafile via cmsrep. Currently I have copied bsunanda to cmsrep , if you have to update the file then please let me know so that I can update it on cmsrep. |
Pull request #8292 was updated. |
Pull request #8292 was updated. |
Pull request #8292 was updated. |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-be770a/30945/summary.html The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:
You can see more details here: Comparison SummarySummary:
|
+externals |
This pull request is fully signed and it will be integrated in one of the next IB/CMSSW_13_1_X/master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @perrotta, @dpiparo, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
@civanch , SL v7 should be available in today's 13.1.X IB |
No description provided.