MxArray: convert cv::Mat of type CV_8U to logical mxArray about mexopencv HOT 5 CLOSED

Very interesting. I'll add special treating for creating mxLOGICAL_CLASS array.

from mexopencv.

The last commit added thresholding to saturate all nonzero values to 1 in the conversion. Hope it fixes the issue.

from mexopencv.

while your commit solves the above case, cv::threshold has a couple of issues:

• it only works with CV_8U, CV_16S, CV_32F types. Anything else throws an exception.
• when converting from signed types, negative values get saturated at zero (false). Note that in MATLAB, logical(-1) == logical(1) == true, and only logical(0) == false

As far as I can tell, a better solution would be:

cv::compare(input, 0, input, cv::CMP_NE); // input = uint8( 255 * (input~=0) )
cv::min(input, 1, input);                 // input = min(input,1)

by comparing the matrix against scalar zero (result is of type CV_8UC1 set to 255 where nonzero), then using cv::min to cap at 1.

Here is a test to confirm (should return: [true,false,true] ie logical([1,0,1])):

void mexFunction(int nlhs, mxArray *plhs[], int nrhs, const mxArray *prhs[])
{
    double x[] = {-5.0, 0.0, 5.0};
    cv::Mat m(1, 3, CV_64FC1, (void*)x);
    plhs[0] = MxArray(m, mxLOGICAL_CLASS);
}

from mexopencv.

That makes sense. I have included the fix in the last commit, with a slight modification of the use of cv::setTo rather than cv::min.

Perhaps it's a TODO to prepare a unit test for MxArray.

from mexopencv.

thanks, that should do it. Unit-testing the MxArray layer sounds like a good idea..

from mexopencv.