-
Notifications
You must be signed in to change notification settings - Fork 0
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
mpfr on docker imager #3
Comments
The current latest docker image has the dependency (https://github.com/rest-for-physics/rest-docker/blob/main/root/dependencies/general). However the framework is not currently using the image defined in this repository for validation due to some other issues. I don't quite remember what was the problem but I think it was related to some other issue that is probably fixed now, hopefully whatever is failing can be fixed easily and we can merge PRs such as rest-for-physics/framework#365. |
It seems the failure at rest-for-physics/framework#365 is due to GEOM_SCALE not found. It seems the pipeline is picking an old Geant4lib version. Since |
I merged rest-for-physics/framework#365 on rest-for-physics/framework#407. So that it uses this docker image. Still it looks like it does not find https://github.com/rest-for-physics/framework/actions/runs/5700965392/job/15451237158?pr=407 Even though it seems to be included on that package:
|
Also the Geant4 tests validation is failing. Perhaps the Geant4 version is not the same? |
I am not sure I follow but the library is installed on the image, you can verify by running:
Version of dependencies can also be queried via:
The version of Geant4 for this image seems to be different that what we are currently using. I will update this image to use the same as the current pipeline, hopefully that fixes it. |
It is clear now that we have to remove
mpreal.h
from the https://github.com/rest-for/physics/framework repository, and simply installsudo apt-get install libmpfrc++-dev
.This is related to the following PR.
rest-for-physics/framework#407
Do we have a docker image with
libmpfrc++-dev
included? Then, plz, @lobis, it is possible to update the docker image at rest-for-physics/framework#407, fix the validation pipeline and fix it?The text was updated successfully, but these errors were encountered: