-
Notifications
You must be signed in to change notification settings - Fork 180
remove lines which inhibit transport at large eta #1797
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
Conversation
|
REQUEST FOR PRODUCTION RELEASES: This will add The following labels are available |
|
The CI seems to fail for this generator ( |
|
It fails also on LXPLUS. Log shows: |
|
Thanks @sawenzel and @jackal1-66, I'm investigating on lxplus, but I think the issue is not related to this PR. It appears for me on lxplus moving from O2sim::v20240703-1 to O2sim::v20240704-1 versions. I'll continue to check, I don't know if you have any hint. Thank you |
|
Dear @sawenzel, I tried to reproduce the error locally, and I see that the ambiguity pointed by the error message can be solved by the following change: @@ -9,7 +9,6 @@ R__ADD_INCLUDE_PATH($O2DPG_ROOT/MC/config/PWGDQ/EvtGen) using namespace o2::eventgen; namespace o2 Do you think this would be fine ? If you agree, I will update this PR accordingly. Thank you |
|
@ffionda : I think this change would be appreciated and a good idea. |
|
@sawenzel: Thanks for confirming. I would like to request to include this PR in the production software with the following labels: |
|
@ffionda : Thanks for the fix! Concerning the tagging: Maybe this is no longer necessary at all. We have now the possibility to load PWG generator configs from an arbitrary tag as outlined in this ticket https://its.cern.ch/jira/browse/O2-5366. The production simply needs to specify a JDL variable ALIEN_JDL_O2DPG_MC_CONFIG_ROOT to point to the correct path on CVFMS. (Or directly set O2DPG_MC_CONFIG_ROOT in small user studies). |
|
@sawenzel: thanks for the input, I think you are right. I tried to set ALIEN_JDL_O2DPG_MC_CONFIG_ROOT (considering the daily O2DPG tag with the fix applied) in the JDL and it seems it works correctly (output tests are available here https://alimonitor.cern.ch/catalogue/?path=%2Falice%2Fcern.ch%2Fuser%2Ff%2Fffionda%2FanchorMC_4317test_NewO2DPG%2Foutput%2F526641#/alice/cern.ch/user/f/ffionda/anchorMC_4317test_NewO2DPG/output/526641 ). |
|
Ported to async-v1-01-02f. |
* remove lines which inhibit transport at large eta * fix Pythia generator ambiguity (cherry picked from commit f55a23a)
* remove lines which inhibit transport at large eta * fix Pythia generator ambiguity
No description provided.