search

serge-sans-paille / pythran

Ahead of Time compiler for numeric kernels
https://pythran.readthedocs.io
BSD 3-Clause "New" or "Revised" License
1.98k stars 191 forks source link

Only link with blas when needed #2207

Closed serge-sans-paille closed 1 month ago

serge-sans-paille commented 1 month ago

Based on the included headers, pythran now automatically removes the blas dependency.

Also do some cleanups (removing debug lines, some formatting, fix imatmul headers).

Fix #2203

serge-sans-paille commented 1 month ago

@martibosch : this one is finally green. It should remove the dep on openblas on your project :-)

martibosch commented 1 month ago

unluckily I got the same error both locally and in github actions :(

martibosch commented 1 month ago

sorry, I made a mistake in the requirement syntax. After fixing, everything is working: https://github.com/martibosch/pylandstats/actions/runs/9077046472

should I release now with the git-branch requirement in pyproject.toml, or should i wait (e.g., maybe you remove feature branches or you plan a tagged release very soon).

Thank you again!

serge-sans-paille commented 1 month ago

On Tue, May 14, 2024 at 03:02:43AM -0700, Martí Bosch wrote:

sorry, I made a mistake in the requirement syntax. After fixing, everything is working: [1]https://github.com/martibosch/pylandstats/actions/runs/9077046472

should I release now with the git-branch requirement in pyproject.toml, or should i wait (e.g., maybe you remove feature branches or you plan a tagged release very soon).

Great! Release incoming this week or the week after :-)