[gdal-dev] MrSID Color Spaces

Wed May 17 08:10:42 PDT 2017

Its an enum value defined in at least version 8.0 and greater.  Maybe in
earlier versions but I am using version 9.1.

Martin

From: Even Rouault [mailto:even.rouault at spatialys.com] 
Sent: Wednesday, May 17, 2017 8:39 AM
To: Martin Chapman
Cc: 'SIMON Nicolas'; gdal-dev at lists.osgeo.org
Subject: Re: [gdal-dev] MrSID Color Spaces

On mercredi 17 mai 2017 08:26:00 CEST Martin Chapman wrote:

> Nicolas,

> 

> 

> 

> Looks good bud! Thanks!

I've just applied it to trunk, but couldn't really test it since it seems my
MrSID version doesn't have those #define

Even

> 

> 

> 

> 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/a893be2b/attachment-0001.html>