Grayscaling Image with GD causes image to turn jet black!

[Phil Powell]

I borrowed this code from a source:

for($a=0;$a<ima gecolorstotal ($image_id);$a+ +)
{
$color = imageColorsForI ndex($image_id, $i);
$R=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$G=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$B=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
imageColorSet($ image_id, $a, $R, $G, $B);
}

using an image (in this case I'm using a JPEG) that is a true color
image I was unable to successfully use imagecopymergeg ray since that
washed out my hues/saturation altogether making the image illegible
(imagecopymerge gray + imagecreate); otherwise, the colors overrode
imagecopymergeg ray (imagecopymerge gray + imagecreatetrue color).

Using the code I borrowed I was able to change the image.. except that
it turned jet black instead!

Has anyone ever successfully grayscaled an image using GD library? If
so, how did you do it?

Thanx
Phil

[CountScubula]

Try running an average of the three colors:
$avg = floor( ( $color['red']) + $color['green']) + $color['blue']) ) / 3 )
$R= $avg; $B = $avg; $G = $avg;
imageColorSet($ image_id, $a, $R, $G, $B);
--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** **@posting.goog le.com...

I borrowed this code from a source:

for($a=0;$a<ima gecolorstotal ($image_id);$a+ +)
{
$color = imageColorsForI ndex($image_id, $i);
$R=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$G=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$B=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
imageColorSet($ image_id, $a, $R, $G, $B);
}

using an image (in this case I'm using a JPEG) that is a true color
image I was unable to successfully use imagecopymergeg ray since that
washed out my hues/saturation altogether making the image illegible
(imagecopymerge gray + imagecreate); otherwise, the colors overrode
imagecopymergeg ray (imagecopymerge gray + imagecreatetrue color).

Using the code I borrowed I was able to change the image.. except that
it turned jet black instead!

Has anyone ever successfully grayscaled an image using GD library? If
so, how did you do it?

Thanx
Phil

[Phil Powell]

I'm sorry but that too breaks. Perhaps a code review is in order, I
realize that imageColorsTota l($this->image) produces 0, which I am
afraid I don't understand:

class ImageGrayscaleG enerator {

/*---------------------------------------------------------------------------------------------------------------------------------------------------------
This class exists due to the rather poor performance of the
imagecopymergeg ray() command
Borrowing code snippet from
http://us3.php.net/manual/en/functio...ymergegray.php first
line comment
Will perform actual grayscaling of image one pixel at a time.
-----------------------------------------------------------------------------------------------------------------------------------------------------------*/

var $image, $image_width, $image_height;

// REMEMBER TO USE REFERENCE POINTER ONTO $image OBJECT TO ENSURE
"static" CHANGE TO OBJECT AND NOT TO INSTANCE

function ImageGrayscaleG enerator(&$imag e) { // CONSTRUCTOR
$this->image =& $image;
}

function makeGray() { // VOID METHOD
print_r($this->image); print_r("<P>");
print_r(@imageC olorsTotal($thi s->image) . "<P>");
for ($i = 0; $i < @imagecolorstot al($this->image); $i++) {
$colorArray = @imageColorsFor Index($this->image, $i);
print_r($colorA rray); print_r("<P>");
$avg = floor(($colorAr ray['red'] + $colorArray['green'] +
$colorArray['blue']) / 3);
@imageColorSet( $this->image, $i, $avg, $avg, $avg);
}
}
}
Thanx
Phil

"CountScubu la" <[e-mail] mail.com> wrote in message news:<_r******* ************@ne wssvr27.news.pr odigy.com>...

Try running an average of the three colors:
$avg = floor( ( $color['red']) + $color['green']) + $color['blue']) ) / 3 )
$R= $avg; $B = $avg; $G = $avg;
imageColorSet($ image_id, $a, $R, $G, $B);
--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** **@posting.goog le.com...

I borrowed this code from a source:

for($a=0;$a<ima gecolorstotal ($image_id);$a+ +)
{
$color = imageColorsForI ndex($image_id, $i);
$R=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$G=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$B=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
imageColorSet($ image_id, $a, $R, $G, $B);
}

using an image (in this case I'm using a JPEG) that is a true color
image I was unable to successfully use imagecopymergeg ray since that
washed out my hues/saturation altogether making the image illegible
(imagecopymerge gray + imagecreate); otherwise, the colors overrode
imagecopymergeg ray (imagecopymerge gray + imagecreatetrue color).

Using the code I borrowed I was able to change the image.. except that
it turned jet black instead!

Has anyone ever successfully grayscaled an image using GD library? If
so, how did you do it?

Thanx
Phil

[CountScubula]

sorry, I didnt really look at the code, how is it breaking, error, not
working?

--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

I'm sorry but that too breaks. Perhaps a code review is in order, I
realize that imageColorsTota l($this->image) produces 0, which I am
afraid I don't understand:

class ImageGrayscaleG enerator {

/*--------------------------------------------------------------------------
----------------------------------------------------------------------------
--- This class exists due to the rather poor performance of the
imagecopymergeg ray() command
Borrowing code snippet from
http://us3.php.net/manual/en/functio...ymergegray.php first
line comment
Will perform actual grayscaling of image one pixel at a time.
-------------------------------------------------------------------------- ----------------------------------------------------------------------------
-----*/
var $image, $image_width, $image_height;

// REMEMBER TO USE REFERENCE POINTER ONTO $image OBJECT TO ENSURE
"static" CHANGE TO OBJECT AND NOT TO INSTANCE

function ImageGrayscaleG enerator(&$imag e) { // CONSTRUCTOR
$this->image =& $image;
}

function makeGray() { // VOID METHOD
print_r($this->image); print_r("<P>");
print_r(@imageC olorsTotal($thi s->image) . "<P>");
for ($i = 0; $i < @imagecolorstot al($this->image); $i++) {
$colorArray = @imageColorsFor Index($this->image, $i);
print_r($colorA rray); print_r("<P>");
$avg = floor(($colorAr ray['red'] + $colorArray['green'] +
$colorArray['blue']) / 3);
@imageColorSet( $this->image, $i, $avg, $avg, $avg);
}
}
}
Thanx
Phil

"CountScubu la" <[e-mail] mail.com> wrote in message

news:<_r******* ************@ne wssvr27.news.pr odigy.com>...

Try running an average of the three colors:
$avg = floor( ( $color['red']) + $color['green']) + $color['blue']) ) / 3 ) $R= $avg; $B = $avg; $G = $avg;
imageColorSet($ image_id, $a, $R, $G, $B);
--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** **@posting.goog le.com...

I borrowed this code from a source:

for($a=0;$a<ima gecolorstotal ($image_id);$a+ +)
{
$color = imageColorsForI ndex($image_id, $i);
$R=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$G=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
$B=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
($color['blue']);
imageColorSet($ image_id, $a, $R, $G, $B);
}

using an image (in this case I'm using a JPEG) that is a true color
image I was unable to successfully use imagecopymergeg ray since that
washed out my hues/saturation altogether making the image illegible
(imagecopymerge gray + imagecreate); otherwise, the colors overrode
imagecopymergeg ray (imagecopymerge gray + imagecreatetrue color).

Using the code I borrowed I was able to change the image.. except that
it turned jet black instead!

Has anyone ever successfully grayscaled an image using GD library? If
so, how did you do it?

Thanx
Phil

[Phil Powell]

"CountScubu la" <[e-mail] mail.com> wrote in message news:<bE******* ***********@new ssvr27.news.pro digy.com>...

sorry, I didnt really look at the code, how is it breaking, error, not
working?

I'll show you, just go down and view...

Phil

--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

I'm sorry but that too breaks. Perhaps a code review is in order, I
realize that imageColorsTota l($this->image) produces 0, which I am
afraid I don't understand:

class ImageGrayscaleG enerator {

/*--------------------------------------------------------------------------
----------------------------------------------------------------------------
---

This class exists due to the rather poor performance of the
imagecopymergeg ray() command
Borrowing code snippet from
http://us3.php.net/manual/en/functio...ymergegray.php first
line comment
Will perform actual grayscaling of image one pixel at a time.
--------------------------------------------------------------------------

----------------------------------------------------------------------------
-----*/

var $image, $image_width, $image_height;

// REMEMBER TO USE REFERENCE POINTER ONTO $image OBJECT TO ENSURE
"static" CHANGE TO OBJECT AND NOT TO INSTANCE

function ImageGrayscaleG enerator(&$imag e) { // CONSTRUCTOR
$this->image =& $image;
}

function makeGray() { // VOID METHOD
print_r($this->image); print_r("<P>");
print_r(@imageC olorsTotal($thi s->image) . "<P>");
** STOP HERE! What happens is that imageColorsTota l($this->image) =
0! Though the image is in color and I do a reference pointer to the
$image resource paramters in the method call, and it actually DOES
change the image according to the value of
imageColorsTota l($this->image), it turns the entire image into one
jet-black image, no tone, no hue, nothing, a big black box.

Phil

for ($i = 0; $i < @imagecolorstot al($this->image); $i++) {
$colorArray = @imageColorsFor Index($this->image, $i);
print_r($colorA rray); print_r("<P>");
$avg = floor(($colorAr ray['red'] + $colorArray['green'] +
$colorArray['blue']) / 3);
@imageColorSet( $this->image, $i, $avg, $avg, $avg);
}
}
}
Thanx
Phil

"CountScubu la" <[e-mail] mail.com> wrote in message

news:<_r******* ************@ne wssvr27.news.pr odigy.com>... Try running an average of the three colors:
$avg = floor( ( $color['red']) + $color['green']) + $color['blue']) ) / 3 ) $R= $avg; $B = $avg; $G = $avg;
imageColorSet($ image_id, $a, $R, $G, $B);
--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** **@posting.goog le.com...
> I borrowed this code from a source:
>
>
>
> for($a=0;$a<ima gecolorstotal ($image_id);$a+ +)
> {
> $color = imageColorsForI ndex($image_id, $i);
> $R=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
> ($color['blue']);
> $G=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
> ($color['blue']);
> $B=.299 * ($color['red'])+ .587 * ($color['green'])+ .114 *
> ($color['blue']);
> imageColorSet($ image_id, $a, $R, $G, $B);
> }
>
> using an image (in this case I'm using a JPEG) that is a true color
> image I was unable to successfully use imagecopymergeg ray since that
> washed out my hues/saturation altogether making the image illegible
> (imagecopymerge gray + imagecreate); otherwise, the colors overrode
> imagecopymergeg ray (imagecopymerge gray + imagecreatetrue color).
>
> Using the code I borrowed I was able to change the image.. except that
> it turned jet black instead!
>
> Has anyone ever successfully grayscaled an image using GD library? If
> so, how did you do it?
>
> Thanx
> Phil

[CountScubula]

"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

I'll show you, just go down and view...

Phil

< Big-O-snip >

Sometimes easier to re-invent the wheel (thats my my cars don't have model-t
wheels on them)
here is some code that works, and you can test here:
http://www.gzentools.com/test/img2gry.php

<?php

$imgFile = "girl1.jpg" ;

$imSrc = imagecreatefrom jpeg($imgFile);

$imW = imagesx($imSrc) ;
$imH = imagesy($imSrc) ;

$imDst = imagecreate($im W,$imH);

// create 256 shades of grey
for ($i=0; $i<256; $i++) {
$colorNdx[$i] = imagecoloralloc ate($imDst,$i,$ i,$i);
}

for ($y=0; $y<$imH; $y++)
{
for ($x=0; $x<$imW; $x++)
{
$ndx = imagecolorat($i mSrc,$x,$y);
$ndxColors = imagecolorsfori ndex($imSrc,$nd x);
$avg = floor( ($ndxColors['red'] + $ndxColors['green'] +
$ndxColors['blue']) / 3 );
imagesetpixel($ imDst,$x,$y,$co lorNdx[$avg]);
}
}

imagejpeg($imDs t);
exit();

?>
--
Mike Bradley
http://www.gzentools.com -- free online php tools

[Phil Powell]

IT WORKED AT LAST!!! Thanx a lot. By the way, your URL produced
ASCII-related garbage on my screen, just so you know.

Thanx again!!
Phil

"CountScubu la" <[e-mail] mail.com> wrote in message news:<vi******* ************@ne wssvr27.news.pr odigy.com>...

"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

I'll show you, just go down and view...

Phil

< Big-O-snip >

Sometimes easier to re-invent the wheel (thats my my cars don't have model-t
wheels on them)
here is some code that works, and you can test here:
http://www.gzentools.com/test/img2gry.php

<?php

$imgFile = "girl1.jpg" ;

$imSrc = imagecreatefrom jpeg($imgFile);

$imW = imagesx($imSrc) ;
$imH = imagesy($imSrc) ;

$imDst = imagecreate($im W,$imH);

// create 256 shades of grey
for ($i=0; $i<256; $i++) {
$colorNdx[$i] = imagecoloralloc ate($imDst,$i,$ i,$i);
}

for ($y=0; $y<$imH; $y++)
{
for ($x=0; $x<$imW; $x++)
{
$ndx = imagecolorat($i mSrc,$x,$y);
$ndxColors = imagecolorsfori ndex($imSrc,$nd x);
$avg = floor( ($ndxColors['red'] + $ndxColors['green'] +
$ndxColors['blue']) / 3 );
imagesetpixel($ imDst,$x,$y,$co lorNdx[$avg]);
}
}

imagejpeg($imDs t);
exit();

?>

[CountScubula]

Like like I forgot to inflate the tire after I reinvented the wheel, forgot
to put:
header("Content-type: image/jpeg");

--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

IT WORKED AT LAST!!! Thanx a lot. By the way, your URL produced
ASCII-related garbage on my screen, just so you know.

Thanx again!!
Phil

"CountScubu la" <[e-mail] mail.com> wrote in message

news:<vi******* ************@ne wssvr27.news.pr odigy.com>...

"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

I'll show you, just go down and view...

Phil

< Big-O-snip >

Sometimes easier to re-invent the wheel (thats my my cars don't have model-t wheels on them)
here is some code that works, and you can test here:
http://www.gzentools.com/test/img2gry.php

<?php

$imgFile = "girl1.jpg" ;

$imSrc = imagecreatefrom jpeg($imgFile);

$imW = imagesx($imSrc) ;
$imH = imagesy($imSrc) ;

$imDst = imagecreate($im W,$imH);

// create 256 shades of grey
for ($i=0; $i<256; $i++) {
$colorNdx[$i] = imagecoloralloc ate($imDst,$i,$ i,$i);
}

for ($y=0; $y<$imH; $y++)
{
for ($x=0; $x<$imW; $x++)
{
$ndx = imagecolorat($i mSrc,$x,$y);
$ndxColors = imagecolorsfori ndex($imSrc,$nd x);
$avg = floor( ($ndxColors['red'] + $ndxColors['green'] +
$ndxColors['blue']) / 3 );
imagesetpixel($ imDst,$x,$y,$co lorNdx[$avg]);
}
}

imagejpeg($imDs t);
exit();

?>

[Phil Powell]

Sorry I can't add that header() line, the moment I do, "index.php"
turns literally into an indiscernible JPEG image (that's assuming I'm
working with a JPEG, in my Image Catalog it could literally be
anything, even a video!)

Phil

"CountScubu la" <[e-mail] mail.com> wrote in message news:<VP******* **********@news svr29.news.prod igy.com>...

Like like I forgot to inflate the tire after I reinvented the wheel, forgot
to put:
header("Content-type: image/jpeg");

--
Mike Bradley
http://www.gzentools.com -- free online php tools
"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...

IT WORKED AT LAST!!! Thanx a lot. By the way, your URL produced
ASCII-related garbage on my screen, just so you know.

Thanx again!!
Phil

"CountScubu la" <[e-mail] mail.com> wrote in message

news:<vi******* ************@ne wssvr27.news.pr odigy.com>...

"Phil Powell" <so*****@erols. com> wrote in message
news:1c******** *************** ***@posting.goo gle.com...
>
> I'll show you, just go down and view...
>
> Phil
>
> < Big-O-snip >

Sometimes easier to re-invent the wheel (thats my my cars don't have model-t wheels on them)
here is some code that works, and you can test here:
http://www.gzentools.com/test/img2gry.php

<?php

$imgFile = "girl1.jpg" ;

$imSrc = imagecreatefrom jpeg($imgFile);

$imW = imagesx($imSrc) ;
$imH = imagesy($imSrc) ;

$imDst = imagecreate($im W,$imH);

// create 256 shades of grey
for ($i=0; $i<256; $i++) {
$colorNdx[$i] = imagecoloralloc ate($imDst,$i,$ i,$i);
}

for ($y=0; $y<$imH; $y++)
{
for ($x=0; $x<$imW; $x++)
{
$ndx = imagecolorat($i mSrc,$x,$y);
$ndxColors = imagecolorsfori ndex($imSrc,$nd x);
$avg = floor( ($ndxColors['red'] + $ndxColors['green'] +
$ndxColors['blue']) / 3 );
imagesetpixel($ imDst,$x,$y,$co lorNdx[$avg]);
}
}

imagejpeg($imDs t);
exit();

?>