-
-
Notifications
You must be signed in to change notification settings - Fork 40
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
openblas v0.3.22 #150
openblas v0.3.22 #150
Conversation
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 ( |
…nda-forge-pinning 2023.03.26.16.10.11
700885b
to
29cd9e7
Compare
@martin-frbg, all unix builds fail with:
Where should this object file be coming from? 🤔 |
interface/zspmv.c - maybe your download of the source tarball was truncated ? |
@martin-frbg, it seems like there are two different code patterns
and
We have |
Eh, thought the unqualified ifdefs had died out already, and no recent change in this function (though the interval for this release was definitely too long) |
Ok, I see I added them myself two months ago :( |
Thank you both! Seems the patch I came up with here is working, so I opened OpenMathLib/OpenBLAS#3972. |
Hm, seems osx-arm64 cross-compilation is unhappy as well...
|
You'll need my 3975 on top of your patch, but it is now looking as if there is something else broken in 0.3.22 - maybe "just" corner cases of ZGETF2 plus a few changed accuracies from LAPACK 3.11.0, maybe more. I'll probably release 0.3.23 as soon as the dust settles. |
now you'd need PR 3980 for the regression in GETF2 as well, may make sense to wait for 0.2.23 unless you want to support checking 0.3.22 for further defects |
I don't mind backporting / testing further until we're sure that all things are sorted out for 0.3.23 (PR 3980 needed 3978 as well). |
osx-arm still unhappy...
|
how come osx-arm is named sandybridge ? Is the |
We didn't change the build instructions since 0.3.21 (where this didn't happen), so either we were implicitly relying on something undocumented, or something broke. |
Well, the build is undoubtedly for an ARM64 system (using kernels from the arm64 subtree and culminating in the success message mentioning libopenblas64_vortexp as the name of the library. But the subsequent |
released 0.3.23 now with the fixes (plus a few Raptor Lake cpuids) |
Cross-compilation still unhappy in #151; I don't have time to look at it this week. |
Superseded by #151 |
It is very likely that the current package version for this feedstock is out of date.
Checklist before merging this PR:
license_file
is packagedInformation about this PR:
@conda-forge-admin,
please add bot automerge
in the title and merge the resulting PR. This command will add our bot automerge feature to your feedstock.bot-rerun
label to this PR. The bot will close this PR and schedule another one. If you do not have permissions to add this label, you can use the phrase@conda-forge-admin, please rerun bot
in a PR comment to have theconda-forge-admin
add it for you.Pending Dependency Version Updates
Here is a list of all the pending dependency version updates for this repo. Please double check all dependencies before merging.
Dependency Analysis
We couldn't run dependency analysis due to an internal error in the bot. :/ Help is very welcome!
This PR was created by the regro-cf-autotick-bot. The regro-cf-autotick-bot is a service to automatically track the dependency graph, migrate packages, and propose package version updates for conda-forge. Feel free to drop us a line if there are any issues! This PR was generated by https://github.com/regro/cf-scripts/actions/runs/4527083694, please use this URL for debugging.