Update python_installation.rst

[willov]

On Python 3.11 and with openblas 0.3.24-2, the amici installer is unable to find "cblas.h". This is fixed by exporting the openblas path to BLAS_CFLAGS

[codecov]

Codecov Report

Merging #2212 (bc9b181) into develop (a41c906) will decrease coverage by 22.71%.
Report is 1 commits behind head on develop.
The diff coverage is n/a.

❗ Current head bc9b181 differs from pull request most recent head 1cdd865. Consider uploading reports for the commit 1cdd865 to get more accurate results

Additional details and impacted files

@@             Coverage Diff              @@
##           develop    #2212       +/-   ##
============================================
- Coverage    76.45%   53.75%   -22.71%     
============================================
  Files           91       38       -53     
  Lines        15028     5659     -9369     
============================================
- Hits         11490     3042     -8448     
+ Misses        3538     2617      -921     

Flag	Coverage Δ
cpp	?
cpp_python	?
petab	53.75% <ø> (-0.03%)	⬇️
python	?
sbmlsuite	?

Flags with carried forward coverage won't be shown. Click here to find out more.

see 81 files with indirect coverage changes

[dweindl]

Thanks @willov.

[willov]

@dweindl, the pull requests was perhaps a bit premature. While it resolves the install issues, there are still problems when trying to import amici. I will get back to you when/if we figure out how to solve the problem.

@Podde1 is also currently looking into the problem.

[dweindl]

@willov : Maybe missing some BLAS_LIBS=-L/path/to/libopenblas.{a,so}?

[willov]

Thanks! You are correct, export BLAS_LIBS="-L/usr/lib/ -lopenblas" solved it in my case (also with a --no-cache-dir). I will check if this also solved the problems for @Podde1, then I will issue another merge request.

[dweindl]

Thanks! You are correct, export BLAS_LIBS="-L/usr/lib/ -lopenblas" solved it in my case (also with a --no-cache-dir). I will check if this also solved the problems for @Podde1, then I will issue another merge request.

👍

I will leave this PR open for now. Please update on this branch. Is -L/usr/lib/ required? I would have expected this directory to be included by default. I.e. is export BLAS_LIBS=-lopenblas not sufficient?
Right, --no-cache-dir is required with recent pip versions in case of linker errors.

[Podde1]

I am have tested this in my environment now aswell. It was indeed sufficient to set export BLAS_LIBS=-lopenblas and exclude -L/usr/lib/.

[willov]

export BLAS_LIBS=-lopenblas was indeed sufficient. I have updated the branch to also include instructions for exporting BLAS_LIBS.