Convert image to grayscale parallel loop

I wrote some code that converts an image to grayscale. but the code only converts partial from it. I am trying to convert this code to parallel computation. I end up with errors that I cannot get around them. Any suggestion?

    private void button2_Click(object sender, EventArgs e)
    {

        Bitmap bmp = (Bitmap)pictureBox1.Image;
        unsafe {
            //get image dimension
            //int width = bmp.Width;
            //int height = bmp.Height;


            BitmapData bitmapData = bmp.LockBits(new Rectangle(0, 0, bmp.Width, bmp.Height), ImageLockMode.ReadWrite, bmp.PixelFormat);

            //define variable
            int bpp = System.Drawing.Bitmap.GetPixelFormatSize(bmp.PixelFormat) / 8;
            int hip = bitmapData.Height;
            int wib = bitmapData.Width + bpp;

            //point to first pixel
            byte* PtrFirstPixel = (byte*)bitmapData.Scan0;
            //color of pixel
           // Color p;

            //grayscale

            Parallel.For(0, hip, y =>
            {
                byte* currentLine = PtrFirstPixel + (y * bitmapData.Stride);
                for (int x = 0; x < wib; x = x + bpp)
                {
                    //get pixel value
                    //p = bmp.GetPixel(x, y);

                    //extract pixel component ARGB
                    //int a = p.A;
                    //int r = p.R;
                    //int g = p.G;
                    // int b = p.B;
                    int b = currentLine[x];
                    int g = currentLine[x + 1];
                    int r = currentLine[x + 2];



                    //find average
                    int avg = (r + g + b) / 3;

                    //set new pixel value
                    // bmp.SetPixel(x, y, Color.FromArgb(a, avg, avg, avg));
                    currentLine[x] = (byte)avg;
                    currentLine[x + 1] = (byte)avg;
                    currentLine[x + 2] = (byte)avg;


                }


            });

            bmp.UnlockBits(bitmapData);



            //load grayscale image in picturebox2
            //pictureBox2.Image = bmp;




        }
        pictureBox2.Image = bmp;

    }

      

my out put image

+3


source to share


1 answer


int wib = bitmapData.Width + bpp;

      

it should be:



int wib = bitmapData.Width * bpp;

      

You want the number of bytes that needs to be multiplied, not added. There may be other problems, but this is definitely not true.

+2


source







All Articles