Skip to content

build for MKL 2025.0#128

Closed
h-vetinari wants to merge 18 commits into
conda-forge:mainfrom
h-vetinari:mkl
Closed

build for MKL 2025.0#128
h-vetinari wants to merge 18 commits into
conda-forge:mainfrom
h-vetinari:mkl

Conversation

@h-vetinari

@h-vetinari h-vetinari commented Nov 8, 2024

Copy link
Copy Markdown
Member

Not having this pushes the solver into weird contortions like

- libblas                            3.9.0  25_win64_mkl           conda-forge        4MB
+ libblas                            3.9.0  25_win64_openblas      conda-forge        4MB
- libcblas                           3.9.0  25_win64_mkl           conda-forge        4MB
+ libcblas                           3.9.0  25_win64_openblas      conda-forge        4MB
- liblapack                          3.9.0  25_win64_mkl           conda-forge        4MB
+ liblapack                          3.9.0  25_win64_openblas      conda-forge        4MB

because it ends up prioritizing the newer MKL; apparently despite the track_features on the non-default BLAS impl here.

We might want to add more track_features here?

@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@h-vetinari

Copy link
Copy Markdown
Member Author

This runs into problems due to conda-forge/libhwloc-feedstock#83 and (I think) wrong metadata in conda-forge/intel_repack-feedstock#82 (that is, unless MKL now requires 2.28 everywhere)

@ZzEeKkAa

Copy link
Copy Markdown

@conda-forge-admin rerender

@ZzEeKkAa

Copy link
Copy Markdown

win64 mkl build is still failing(

@ZzEeKkAa

Copy link
Copy Markdown

@mkrainiuk could you take a look?

@conda-forge-admin

conda-forge-admin commented Dec 17, 2024

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

I do have some suggestions for making it better though...

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/13401950788. Examine the logs at this URL for more detail.

@h-vetinari

Copy link
Copy Markdown
Member Author

Did some digging where the use of mkl_rt comes from - it's been there since the very beginning of windows support: bf90dd3

Furthermore, the MKL docs indicate that this should still work. I've asked again in conda-forge/intel_repack-feedstock#83

@h-vetinari h-vetinari marked this pull request as draft February 14, 2025 04:42
@h-vetinari h-vetinari force-pushed the mkl branch 2 times, most recently from cdcc629 to bdc9e21 Compare February 14, 2025 07:51
antonwolfy added a commit to IntelPython/dpnp that referenced this pull request Feb 18, 2025
There is currently no way to install dpnp with any numpy version > 2.0,
because
- there is no such version available on
`https://software.repos.intel.com/python/conda/` channel
- numpy from `conda-forge` channel depends on BLAS libraries which has a
runtime dependency `mkl < 2025.0` (since BLAS packages with build number
29)

There is draft PR
([gh-128](conda-forge/blas-feedstock#128)) to
build BLAS with MKL version 2025.0, but it is blocked due to MKL issue
([gh-83](conda-forge/intel_repack-feedstock#83)).

Meanwhile there was conda-forge patch
([gh-959](conda-forge/conda-forge-repodata-patches-feedstock#959))
merged recently to add `mkl < 2025.0` constraint to already published
BLAS packaged on `conda-forge` with older build number.

DPNP package can't fulfill the requirement `mkl < 2025.0`, because it
would mandate dpnp build pinning with DPC++ compiler 2024.2. While there
were non-backward compatible changes between 2024.2 and 2025.x versions,
that is dpnp built with 2024.2 compiler can not be installed with any
DPC++ RT 2025.x version, but it is required since targeted by the
upcoming release.

Thus the PR proposes to temporary disable or workaround all scenario
where it is need to tests with numpy from `conda-forge` channel,
including tests workflows for python 3.13 (per PR trigger and nightly)
and workflow for testing of oneMath interfaces with the latest numpy.
The changes has to be rolled back once the above issues are resolved.
github-actions Bot added a commit to IntelPython/dpnp that referenced this pull request Feb 18, 2025
There is currently no way to install dpnp with any numpy version > 2.0,
because
- there is no such version available on
`https://software.repos.intel.com/python/conda/` channel
- numpy from `conda-forge` channel depends on BLAS libraries which has a
runtime dependency `mkl < 2025.0` (since BLAS packages with build number
29)

There is draft PR
([gh-128](conda-forge/blas-feedstock#128)) to
build BLAS with MKL version 2025.0, but it is blocked due to MKL issue
([gh-83](conda-forge/intel_repack-feedstock#83)).

Meanwhile there was conda-forge patch
([gh-959](conda-forge/conda-forge-repodata-patches-feedstock#959))
merged recently to add `mkl < 2025.0` constraint to already published
BLAS packaged on `conda-forge` with older build number.

DPNP package can't fulfill the requirement `mkl < 2025.0`, because it
would mandate dpnp build pinning with DPC++ compiler 2024.2. While there
were non-backward compatible changes between 2024.2 and 2025.x versions,
that is dpnp built with 2024.2 compiler can not be installed with any
DPC++ RT 2025.x version, but it is required since targeted by the
upcoming release.

Thus the PR proposes to temporary disable or workaround all scenario
where it is need to tests with numpy from `conda-forge` channel,
including tests workflows for python 3.13 (per PR trigger and nightly)
and workflow for testing of oneMath interfaces with the latest numpy.
The changes has to be rolled back once the above issues are resolved. b09533e
@stijn-vanesch

Copy link
Copy Markdown

What is the status of mkl >=2025 support? This is currently blocking for us as pytorch with xpu support (provided on intel channel) requires mkl>=2025 and opencv requires libblas

…5.10.28.12.55.54

Other tools:
- conda-build 25.9.0
- rattler-build 0.48.1
- rattler-build-conda-compat 1.4.9
@conda-forge-admin

Copy link
Copy Markdown
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.

Here's what I've got...

For recipe/meta.yaml:

  • ❌ You're setting a constraint on the __osx virtual package directly; this should now be done by adding a build dependence on {{ stdlib("c") }}, and overriding c_stdlib_version in recipe/conda_build_config.yaml for the respective platform as necessary. For further details, please see META: {{ stdlib("c") }} migration conda-forge.github.io#2102.

For recipe/meta.yaml:

  • ℹ️ The recipe is not parsable by parser conda-souschef (grayskull). This parser is not currently used by conda-forge, but may be in the future. We are collecting information to see which recipes are compatible with grayskull.
  • ℹ️ The recipe is not parsable by parser conda-recipe-manager. The recipe can only be automatically migrated to the new v1 format if it is parseable by conda-recipe-manager.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/18876476385. Examine the logs at this URL for more detail.

@mgorny mgorny mentioned this pull request Oct 28, 2025
5 tasks
@h-vetinari

Copy link
Copy Markdown
Member Author

Almost a year later, the problems seem to have simply gone away... #150

@h-vetinari h-vetinari closed this Oct 28, 2025
@h-vetinari h-vetinari deleted the mkl branch October 28, 2025 21:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants