openmp builds require openblas openmp builds

[minrk]

From the openblas faq:

OpenMP provides its own locking mechanisms, so when your code makes BLAS/LAPACK calls from inside OpenMP parallel regions it is imperative that you use an OpenBLAS that is built with USE_OPENMP=1, as otherwise deadlocks might occur.

This is likely the cause of #125 because the pthreads build of openblas is the default blas on linux, but any package that calls blas from within openmp must require openblas to also use openmp.

hopefully closes #125

[conda-forge-webservices]

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.

[traversaro]

Thanks for the insight! I am currently travelling but I will check if this solves the problem as soon as I have access to a laptop.

[traversaro]

Thanks, I just tested this in ami-iit/bipedal-locomotion-framework#886 and indeed it solves the problem!

[minrk]

Great, thank you!

[traversaro]

@minrk Could it make sense to mark mumps-seq-5.7.3=*_3 as broken? I have CI (in ami-iit/bipedal-locomotion-framework#884) that still fail as somehow they are installing the _3 build:

2024-09-17T07:56:08.5855551Z     mumps-include-5.7.3        |       ha770c72_4          22 KB  conda-forge
2024-09-17T07:56:08.5856257Z     mumps-seq-5.7.3            |       h96124f5_3         2.3 MB  conda-forge

and still results in the hanging problem that is quite difficult to detect early on. If you agree, I can prepare the PR to the repo to ask for marking packages as broken.

[minrk]

I will backport the constraint to the repodata patches

[minrk]

repodata patch: conda-forge/conda-forge-repodata-patches-feedstock#847

[minrk]

btw, it's possible older builds may be getting picked due weighing down the openmp build via features. Related PR to try to address the inappropriate blas priority:

• remove track_features for openmp openblas-feedstock#164
• weigh down netlib with an extra feature lapack-feedstock#71