search

BSI-OFIQ / OFIQ-Project

Open Source Facial Image Quality
Other
38 stars 12 forks source link

OpenCV error is back in the latest version #41

Closed tadam98s closed 1 month ago

tadam98s commented 1 month ago

OpenCV error is back in the latest version. The error dows not happen with my previous download from 2024-05-15.

Also, PLEASE do not abort when there is an error and just write an error line in the CSV for this filename. I am running on thousands of images and it is a nightmare when it aborts due to a missing simple try-catch. I rever back to the previous version. (I am unable to share the image)

OpenCV: terminate handler is called! The last OpenCV error is: OpenCV(4.5.5) Error: Image step is wrong (The matrix is not continuous, thus its number of rows can not be changed) in cv::Mat::reshape, file C:\Users\Mickey.conan2\p\b\openc00b993b028ed5\b\src\modules\core\src\matrix.cpp, line 1176

What is strange is that I saved the image with paint to bmp. closed paint. Opened the bmp and saved as jpg and the error is still there. Image dimetions are 592x500 (widthxheight)

Best, Mickey

m-schaubert commented 1 month ago

Hi Mickey, we've committed a fix for "The matrix is not continuous ..." error, could you please check out and test the latest version using that one "not continuous" image. Thanks, Maxim

tadam98s commented 1 month ago

You did in the version of 2024-05-15. But what I dowloaded after the latest fix to the installation procedure + MAC on 2024-06-20 revived the error. So I reverted back to 2024-05-15. Please check the latest version maybe it did not include the correction.

m-schaubert commented 1 month ago

It turned out, that we had this issue in two places, and the second place was "activated" with the commit on 2024-06-20. So we fixed that yesterday: https://github.com/BSI-OFIQ/OFIQ-Project/pull/37/commits/263bef73ac3da469e59c9d27ac09badc17fe5488

tadam98s commented 1 month ago

I can confirm the problem is resolved.

best, mickey