followup on scipy breaking

conda-forge / openblas-feedstock

A conda-smithy repository for openblas.

BSD 3-Clause "New" or "Revised" License

9 stars 39 forks source link

followup on scipy breaking #147

Closed ngam closed 2 years ago

ngam commented 2 years ago

https://github.com/conda-forge/openblas-feedstock/pull/146

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase code>@<space/conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

conda-forge-linter commented 2 years ago

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) and found it was in an excellent condition.

ngam commented 2 years ago

@conda-forge-admin, please rerender

ngam commented 2 years ago

@mkitti

mkitti commented 2 years ago

Is the plan that we merge this and then see if scipy starts working again?

ngam commented 2 years ago

Is the plan that we merge this and then see if scipy starts working again?

No, we can just it locally/separately. Download artifacts, etc. unless people want to merge this...

rgommers commented 2 years ago

What is the best way to download and install a package built in this PR?

h-vetinari commented 2 years ago

What is the best way to download and install a package built in this PR?

Needs the following in conda-forge.yml plus a rerender:

# TODO: remove after debug
azure:
  store_build_artifacts: true

ngam commented 2 years ago

What is the best way to download and install a package built in this PR?

Needs the following in conda-forge.yml plus a rerender:
# TODO: remove after debug
azure:
  store_build_artifacts: true

Already done.

What is the best way to download and install a package built in this PR?

Here: https://dev.azure.com/conda-forge/feedstock-builds/_build/results?buildId=562537&view=artifacts&pathAsName=false&type=publishedArtifacts (hover over and go to the three vertical dots on the very right to download)

ngam commented 2 years ago

Once downloaded you can either:

upload to a personal channel and then pull down like usual with -c yourchannel
install directly from the path, mamba install thepathtobz2

Not if you install directly, it won't install the deps automatically (I usually install the latest from conda-forge, then install a local one)

ngam commented 2 years ago

I will test arm64 for you now

ngam commented 2 years ago

builds are available https://anaconda.org/ngam/openblas/files

ngam commented 2 years ago

Matrix isolating the errors on osx-64: https://github.com/ngam/test_openblas_scipy/

Conclusion: as rgommers said, the errors are due to fortran updates (or more precisely, the automated updates in this PR).

h-vetinari commented 2 years ago

Already done.

Sorry I didn't check this (was on a phone...), seems you're well on top of things. ;)

Once downloaded you can either:

upload to a personal channel and then pull down like usual with -c yourchannel

Note that you can use

conda install -c /path/to/where/you/unpacked/the/artefact openblas

if you download it directly.

h-vetinari commented 2 years ago

Conclusion: as rgommers said, the errors are due to fortran updates (or more precisely, the automated updates in this PR).

Do we have any idea/intuition why a compiler upgrade would break this (well, aside from compiler regressions)...?

ngam commented 2 years ago

No idea, back to @rgommers to evaluate...

ngam commented 2 years ago

Closing since this issue is nothing to do with our intentional updates and it is just compilers, so upstream will fix.