Config.py Failing After Sending Wrong Type to cdll.LoadLibrary()

[Walter3ffb]

In line 285 of config.py, cdll.LoadLibrary takes mkl_path as an argument,
mkl = cdll.LoadLibrary(mkl_path)

mkl_path is a Path object, where LoadLibrary only accepts strings.

This leads to MCEq failing to run if the path to mkl exists (at least on Windows 10).

This should be fixable by sending mkl_path as a string.
mkl = cdll.LoadLibrary(str(mkl_path))

[StFroese]

Thanks @Walter3ffb for opening the issue, I'll check it out!

[StFroese]

@Walter3ffb so the easiest fix on your side to make it run right now is to upgrade to a newer version of Python (>=3.12). I assume you are on 3.11 or earlier.
The problem you encounter has been fixed with 3.12, see here python/cpython#7032.

Currently anything <=3.9 is worth upgrading (https://devguide.python.org/versions/)

@afedynitch If we want to support older python versions with windows we should add the line that is used in the 3.12 fix to the config script:

import os
mkl_path = os.fspath(mkl_path)

[Walter3ffb]

@StFroese Thanks for the reminder; it's been a while since I upgraded!

[Walter3ffb]

After upgrading to the newest version of python, everything works fine

[afedynitch]

Doesn't CI catch this? If not, we should test all official python, OS and arch versions.

Edit: we likely don't test any mkl stuff....

[StFroese]

@afedynitch Or should we orientated at this https://devguide.python.org/versions/ ? So all above 3.9?
CI tests are running on 3.12 with mkl.
I'll add the line from above to the config, so it in principle should work for all python versions :)

[afedynitch]

We should test on all release versions of python from 3.9 to 3.14. 3.9 is technically EOL but I guess that it is still relatively widely used.

We will remove 3.9 once numpy starts dropping it from binary builds.

Also, we should test on Windows latest, Macos 13 and latest, Ubuntu x86 and Ubuntu ARM.