-
Notifications
You must be signed in to change notification settings - Fork 21
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
add decay files for UpsilonXS #24
Conversation
Following a request of a new Upsilon MC production with fixed feed-down
A new Pull Request was created by @michael-pitt (Michael Pitt) for branch master. @aandvalenzuela, @SiewYan, @smuzaffar, @alberto-sanchez, @bbilin, @cmsbuild, @mkirsano, @GurpreetSinghChahal, @iarspider, @menglu21 can you please review it and eventually sign? Thanks. |
Hello Experts, Could you please review it and probably merge it? |
@michael-pitt @tanmayvb , is there any cmssw PR which makes use of these new data files? |
Hi @smuzaffar, A new MC production request will use these new data files. In the Pythia fragment, we can read decay tables provided by users. For example, in our case, we are using: SMP-RunIISummer20UL16GEN-00050 |
@michael-pitt , so by default it will go in to 14.0.X release cycle. Should we backport it to other release cycle(s)? |
please test |
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-794b04/36298/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 |
These files will be accessed after setting up the 10.6.X release used in Run2 production campaigns. |
I will merge it for 14.0.X as soon as any generator L2 signs it |
+1 |
This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2) |
Hi @smuzaffar what is needed to make these files available in the release? I tried to run production code that uses these files, but I received the following error: |
Hi @antoniovilela @sextonkennedy @rappoccio can you check this PR? we are waiting for this PR to be merged to start a new MC production |
+1 |
Hi all, Thank you very much. Best regards, |
Following a request of a new Upsilon MC production with fixed feed-down, and consulting with this readme file.
Regards,
Michael