54

I'm trying to set a new color value to some pixel into a cv::Mat image my code is below:

    Mat image = img;
    for(int y=0;y<img.rows;y++)
    {
        for(int x=0;x<img.cols;x++)
        {
        Vec3b color = image.at<Vec3b>(Point(x,y));
        if(color[0] > 150 && color[1] > 150 && color[2] > 150)
        {
            color[0] = 0;
            color[1] = 0;
            color[2] = 0;
            cout << "Pixel >200 :" << x << "," << y << endl;
        }
        else
        {
            color.val[0] = 255;
            color.val[1] = 255;
            color.val[2] = 255;
        }
    }
    imwrite("../images/imgopti"+to_string(i)+".tiff",image);

It seems to get the good pixel in output (with cout) however in the output image (with imwrite) the pixel concerned aren't modified. I have already tried using color.val[0].. I still can't figure out why the pixel colors in the output image dont change. thanks

3 Answers 3

90

You did everything except copying the new pixel value back to the image.

This line takes a copy of the pixel into a local variable:

Vec3b color = image.at<Vec3b>(Point(x,y));

So, after changing color as you require, just set it back like this:

image.at<Vec3b>(Point(x,y)) = color;

So, in full, something like this:

Mat image = img;
for(int y=0;y<img.rows;y++)
{
    for(int x=0;x<img.cols;x++)
    {
        // get pixel
        Vec3b & color = image.at<Vec3b>(y,x);

        // ... do something to the color ....
        color[0] = 13;
        color[1] = 13;
        color[2] = 13;

        // set pixel
        //image.at<Vec3b>(Point(x,y)) = color;
        //if you copy value
    }
}
0
21

just use a reference:

Vec3b & color = image.at<Vec3b>(y,x);
color[2] = 13;
1
  • 2
    because the problem was about a color Mat. in the uchar case, ofc. use uchar& c = image.at<uchar>(y,x)
    – berak
    Aug 2, 2014 at 14:54
17

I would not use .at for performance reasons.

Define a struct:

//#pragma pack(push, 2) //not useful (see comments below)
struct BGR {
    uchar blue;
    uchar green;
    uchar red;  };

And then use it like this on your cv::Mat image:

BGR& bgr = image.ptr<BGR>(y)[x];

image.ptr(y) gives you a pointer to the scanline y. And iterate through the pixels with loops of x and y

7
  • What is the sense of setting pack value to 2 if sizeof(uchar) is 1? Aug 2, 2014 at 16:01
  • 1
    The compiler will add padding Bytes to a struct. The setting of pack changes those paddings so that the data is aligned in another way.Edit: See this post stackoverflow.com/questions/3318410/pragma-pack-effect
    – Flocke
    Aug 3, 2014 at 7:34
  • 3
    pragma has no effect on a struct made solely from uchars. Read a nice explanation here for details. Aug 3, 2014 at 15:16
  • Thx good link. Seem the pragma really is useless in this case :) thx!
    – Flocke
    Aug 3, 2014 at 16:57
  • You're welcome :) And nice answer: less magic, more efficient. Aug 3, 2014 at 17:50

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Not the answer you're looking for? Browse other questions tagged or ask your own question.