3.12.1#152
Conversation
|
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 ( Here's what I've got... For recipe/meta.yaml:
For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/19622705669. Examine the logs at this URL for more detail. |
|
Please increment blas_minor for 3.12 by 100 |
|
Done locally. I'm working on BLIS bump first, then I'll push to see if that improved anything. |
Could you rebase this PR on main, even without the blis stuff? |
|
Done. |
|
Not looking too good for 3.12 for now; we'll definitely need a blis update, but also MKL does not look compatible yet (missing symbols and thus failed tests) |
|
For expectation management, I had been trying to update our lapack stack past 3.9.0 since almost 5 years ago, so don't be surprised if this takes a while. Before running into the MKL SDL failures, the longest wait was for MKL to become compatible at all -- I recommend asking for an update in conda-forge/intel_repack-feedstock#26 |
|
Am I also seeing correctly that Accelerate is quite incompatible too? I guess that's going to be a hard blocker, unless we want to proceed with the alternate approach of per-platform LAPACK versions. |
|
newaccelerate should be compatible with 3.12 (as of macOS 15.5, c.f #158), but it's possible that that's only for 3.12.0 and not 3.12.1 |
|
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 ( I do have some suggestions for making it better though... For recipe/meta.yaml:
This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/26493188258. Examine the logs at this URL for more detail. |
|
Ok, I think the problem with BLIS is that |
Disable building INDEX64_EXT_API, as it includes ILP64 symbols within the LP64 library. We cannot do that unless all the alternative providers do that as well, as otherwise packages can start linking to these symbols. We already hit this in the wild while trying to test BLIS with LAPACK 3.12.1. See: conda-forge/blas-feedstock#152 (comment) Signed-off-by: Michał Górny <mgorny@quansight.com>
|
@h-vetinari, looks like it's pulling |
I can only guess that something is going wrong with the fact that those builds were marked broken on a non-main label; I've removed the |
|
Hmm, still something suspicious: |
|
@h-vetinari, could you also remove the label from |
now removed from all the associated outputs of that build. Not sure why that didn't occur to me yesterday, sorry |
|
Ok, BLIS looks good now. What remains:
|
Rebase the 2/3 patches. Backport fixes for a bunch of upstream mistakes in Fortran code (i.e. everything I needed in Gentoo before). Signed-off-by: Michał Górny <mgorny@quansight.com>
Signed-off-by: Michał Górny <mgorny@quansight.com>
This reverts commit 0a9bf52.
…6.05.26.15.19.28 Other tools: - conda-build 26.5.0 - rattler-build 0.65.0 - rattler-build-conda-compat 1.4.14
|
Looks like on win+MKL, resp for win+openblas; the old accelerate will not be recoverable, but that's fine and can IMO be dropped. |
Checklist
0(if the version changed)conda-smithy(Use the phrase@conda-forge-admin, please rerenderin a comment in this PR for automated rerendering)Based on #151.