[gdal-dev] MrSID Color Spaces

[Martin Chapman]

Nicolas,

 

Looks good bud!  Thanks!

 

Best regards,

Martin

 

From: SIMON Nicolas [mailto:nicolas.simon at spw.wallonie.be] 
Sent: Wednesday, May 17, 2017 4:41 AM
To: Martin Chapman
Cc: gdal-dev at lists.osgeo.org; Even Rouault
Subject: RE: [gdal-dev] MrSID Color Spaces

 

Martin,

Could you test this patch ?

Nicolas

 

De : gdal-dev [mailto:gdal-dev-bounces at lists.osgeo.org] De la part de Even
Rouault
Envoyé : mardi 16 mai 2017 18:28
À : Martin Chapman
Cc : gdal-dev at lists.osgeo.org
Objet : Re: [gdal-dev] MrSID Color Spaces

 

On mardi 16 mai 2017 10:13:52 CEST Martin Chapman wrote:

> Even or Frank or whoever it may concern,

> 

> 

> 

> Not a big deal but I was trying to open a JP2 file using the MrSID driver

> and noticed that the mrsiddataset.cpp does not identify the color spaces

> MRSID_LTI_COLORSPACE_GRAYSCALEA and MRSID_LTI_COLORSPACE_GRAYSCALEA_PM

> configurations in the MrSIDRasterBand() constructor. I happen to have a
two

> band JP2 that is a grayscale image with an alpha channel. The color

> interpretation comes back as GCI_undefined. Not that important but if you

> ever feel like fixing it, that would be nice.

> 

> 

> 

> The fix would go somewhere close to line 485 in mrsiddataset.cpp.
Basically

> just need to add a couple of more case statements in the switch statement.

 

Martin,

 

Seems you are closed to a patch. Can you provide it ?

 

Even

 

-- 

Spatialys - Geospatial professional services

http://www.spatialys.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.osgeo.org/pipermail/gdal-dev/attachments/20170517/278b1443/attachment.html>