Update Dockerfile to avoid the additional clone #45
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
I just tried to build this docker image and had some issues. This is with
install_apt_packages(cfg)
andinstall_libraries_from_source(cfg)
commented out (i.e. a pure Python development environment). I'm using an AMD machine running Fedora 41. The issues are mostly due to numpy compilation failing for the following reasons, and their resolutions:gfortran
to Dockerfileintel-mkl
,libatlas-base-dev
,liblapack-dev
, andlibblas-dev
to Dockerfilenumpy==1.20.2
contained some compilation issues, and was duplicated twice in therequirements.txt
-> removed entirely fromrequirements.txt
and install before other packages withincreate_venv.sh
. The version is no longer specified.Additionally
EVO
package required the toolunzip
-> addedunzip
to Dockerfiledevelop
, none of my changes were taking effect -> I added the ability to run the local copy, allowing me to build with changes that haven't been pushed toorigin
yet. This requires the Dockerfile to be moved to the root directory of the repo, due to howdocker build
works.Here is proof of a successful build with

install_apt_packages(cfg)
andinstall_libraries_from_source(cfg)
commented out:And with them uncommented (the full build in 3106.5s):

Notes:
ENV MKL_DEBUG_CPU_TYPE=5
to the Dockerfile. Apparentlyintel-mkl
leaves some performance left on the table when using an AMD CPU. This env flag ameliorates that but may not be necessary. It may be possible to make this conditional on the CPU of the current machine.I understand if you do not want to merge these changes. However, if you do want these changes with some modifications, please let me know and I'll be happy to make them. Thanks for your work!