-
Notifications
You must be signed in to change notification settings - Fork 772
{lib,vis}[foss/2023b] OpenImageIO v3.0.6.1, OpenCV v4.11.0 w/ contrib #23523
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
{lib,vis}[foss/2023b] OpenImageIO v3.0.6.1, OpenCV v4.11.0 w/ contrib #23523
Conversation
…foss-2023b-contrib.eb
Updated software
|
|
Can you please add OpenColorIO-2.4.2-foss-2023b.eb |
|
I added it in a seperate pr see #23522 |
…asyconfigs into 20250729182428_new_pr_OpenImageIO3061
|
@boegelbot please test @ jsc-zen3 |
|
Test report by @Crivella |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3163834349 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @boegelbot |
For some arcane reason on my build it the path to the hal_internal.cpp...
#include "precomp.hpp"
#include "hal_internal.hpp"
...opencv_lapack.h...
#include "/home/crivella/.local/easybuild/software/OpenBLAS/0.3.24-GCC-13.2.0/include/cblas.h"
#include "/home/crivella/.local/easybuild/software/OpenBLAS/0.3.24-GCC-13.2.0/include/lapacke.h"
...from the configure step and the OpenBLAS include file does not seem to have the generic definitions crivella@crivella-desktop:~/.local/easybuild/software/OpenBLAS/0.3.24-GCC-13.2.0/include$ grep -R "cblas_zgem" *
cblas.h:void cblas_zgemv64_(OPENBLAS_CONST enum CBLAS_ORDER order, OPENBLAS_CONST enum CBLAS_TRANSPOSE trans, OPENBLAS_CONST blasint m, OPENBLAS_CONST blasint n,
cblas.h:void cblas_zgemm64_(OPENBLAS_CONST enum CBLAS_ORDER Order, OPENBLAS_CONST enum CBLAS_TRANSPOSE TransA, OPENBLAS_CONST enum CBLAS_TRANSPOSE TransB, OPENBLAS_CONST blasint M, OPENBLAS_CONST blasint N, OPENBLAS_CONST blasint K,
cblas.h:void cblas_zgemm3m64_(OPENBLAS_CONST enum CBLAS_ORDER Order, OPENBLAS_CONST enum CBLAS_TRANSPOSE TransA, OPENBLAS_CONST enum CBLAS_TRANSPOSE TransB, OPENBLAS_CONST blasint M, OPENBLAS_CONST blasint N, OPENBLAS_CONST blasint K,Weirdly this is different if i do it on the same OpenBLAS from EESSI Gonna try and redo a rebuild of mine just in case |
|
Umh nope still the same my guess here is that the multiple builds of OpenBLAS in order to also build 64 bit support that are run with The builds on EESSI do not have the So we have to either figure out how to get FlexiBLAS to be used, or figure out if we should switch the order the libraries are build in the easyblock, or alternatively ensure that we get not only multiple version of the library, but also of the include files? @bartoldeman Do you know if the |
|
BTW beside trying to get OpenCV to pick-up the FlexiBLAS include path instead of the OpenBLAS one might be the only thing pertaining this PR, the rest of the discussion is more OpenBLAS specific |
|
Test report by @laraPPr |
|
Test report by @laraPPr |
|
And from the Also not sure FlexiBLAS is explicitly supported https://github.com/opencv/opencv/blob/31b0eeea0b44b370fd0712312df4214d4ae1b158/cmake/OpenCVFindLAPACK.cmake#L166 Have to check if we can manually pass |
|
So the problem i am having will only appear if you have a recent build of OpenBLAS after the merging of Since before the 64bit version was not being built this was not a problem. I've opened an easyblock Issue and a PR that should fix this by manually telling to OpenCV the lapack details if FlexiBLAS is found |
|
Test report by @Crivella |
|
From the configure step It seems BOOST is not used by this package anymore it seems since 3.0.0.3 |
easybuild/easyconfigs/o/OpenImageIO/OpenImageIO-3.0.6.1-foss-2023b.eb
Outdated
Show resolved
Hide resolved
Co-authored-by: Davide Grassano <[email protected]>
|
@boegelbot please test @ jsc-zen3 |
|
@Crivella: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3371103108 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Crivella |
|
Test report by @boegelbot |
Crivella
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
|
Going in, thanks @laraPPr! |
(created using
eb --new-pr)