-
Notifications
You must be signed in to change notification settings - Fork 166
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
Compile starlight against dpmjet #5680
base: master
Are you sure you want to change the base?
Conversation
There might be an issue in building this (I tested it locally). I will start the CI to see if it appears also there. |
It needs this: alisw/DPMJET#7 |
I included the DPMJET PR in my build, but I overlooked the STARlight patch you suggested. Indeed it doesn't build without that. Going to check now the suggested changes |
I made some more tests. Your patch does not work with our current STARlight version, but that's not a big problem because we can update it to the last one available on STARlightsim. The modification for DPMJET as a shared library can be suggested directly as a PR in the main repository (it's probably a good idea to have it there). For the fact that STARlight is not able to find the library instead I'm trying to find alternative solutions, maybe a modification in the recipe will be required. |
I can send the PR to STARlightsim anytime (developers are our colleagues from ALICE). I have there the library build option for the time being, let me know if anything else is needed. |
I discussed with @ktf and there's an easy way to do the second point:
|
|
STARlight can be compiled against DPMJET, in that case a separate library is created (libDpmJetLib) that uses routines from DPMJET to simulate photo-nuclear interactions. UPC group would like to use this mode.
The libDpmJetLib is somehow not installed and remains in the build directory so I added explicit line to move it, maybe there is another solution.
There is an adjustment in STARlight itself needed to make this work. I can make the PR myself, but I will appreciate an expert opinion on issue 2.
STARlight.patch