[JPP-Devel] WMS Layer / SID Layer

2013-04-14 Thread Michaël Michaud
Hi Stefan, Larry

While updating WMS code, I've found several patches from Stefan
about SID Layer handling.
Seems that SIDLayer inherits from WMS Layer, hence, patches
have been added in WMS code to exclude cases where WMS Layers
are instances of SIDLayer.

The design seems error prone and probably currently buggy (and
maybe more now that I've updated the code with WMS only in
mind).
Maybe the model would need an abstract parent for both WMS
and SID Layer.
The reason of sharing is probably the common createImage()
method, but WMSLayer has also references to the WMService
which have nothing to do with SID Layer (AFAIK).

Agree with that ? Any comment ?

Regards,

Michaël

--
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread Larry Becker
HI Michaël,

  Yes, you are correct.  I believe his is not the first time changes to
WMSLayer have affected SIDLayer.  The original SIDLayer was a quick hack
that never got the refactoring it deserved.  A common abstract parent is
probably the best practice.

  However, I'm not sure why OJ still has the SIDLayer class since it was
incorporated into the last version of vividsolutions JUMP as
MrSIDReferencedImage (which is not descended from WMSLayer)..  Is there
some feature of the original class that isn't in it? The java2xml perhaps
doesn't support saving to the project as well?   I would like to test this,
but after an hour of trying I was unable to get a MRSID file to load in
OJ.  There are many different wiki pages and readme files that contradict
each other, but I couldn't get any of them to work.  I finally resorted to
debug and found the required path for the support executables, however I
got an NPE when I tried to open a SID file  at
com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
This is the version that doesn't depend on WMSLayer so something else is
going on, but I don't have time right now to trace it down.

The path to the executables is:   ../lib/ext/etc/  for mrsidgeodecode.exe
and mrsidgeoinfo.exe, and also a tmp folder for the tiles.

regards,

Larry

On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud
wrote:

> Hi Stefan, Larry
>
> While updating WMS code, I've found several patches from Stefan
> about SID Layer handling.
> Seems that SIDLayer inherits from WMS Layer, hence, patches
> have been added in WMS code to exclude cases where WMS Layers
> are instances of SIDLayer.
>
> The design seems error prone and probably currently buggy (and
> maybe more now that I've updated the code with WMS only in
> mind).
> Maybe the model would need an abstract parent for both WMS
> and SID Layer.
> The reason of sharing is probably the common createImage()
> method, but WMSLayer has also references to the WMService
> which have nothing to do with SID Layer (AFAIK).
>
> Agree with that ? Any comment ?
>
> Regards,
>
> Michaël
>
>
> --
> Precog is a next-generation analytics platform capable of advanced
> analytics on semi-structured data. The platform includes APIs for building
> apps and a phenomenal toolset for data science. Developers can use
> our toolset for easy data analysis & visualization. Get a free account!
> http://www2.precog.com/precogplatform/slashdotnewsletter
> ___
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
--
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread edgar . soldin
is MrSid a format still in use? should we fix it? .. ede

On 15.04.2013 16:46, Larry Becker wrote:
> HI Michaël,
> 
>   Yes, you are correct.  I believe his is not the first time changes to 
> WMSLayer have affected SIDLayer.  The original SIDLayer was a quick hack that 
> never got the refactoring it deserved.  A common abstract parent is probably 
> the best practice. 
> 
>   However, I'm not sure why OJ still has the SIDLayer class since it was 
> incorporated into the last version of vividsolutions JUMP as 
> MrSIDReferencedImage (which is not descended from WMSLayer)..  Is there some 
> feature of the original class that isn't in it? The java2xml perhaps doesn't 
> support saving to the project as well?   I would like to test this, but after 
> an hour of trying I was unable to get a MRSID file to load in OJ.  There are 
> many different wiki pages and readme files that contradict each other, but I 
> couldn't get any of them to work.  I finally resorted to debug and found the 
> required path for the support executables, however I got an NPE when I tried 
> to open a SID file  at 
> com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
>   This is the version that doesn't depend on WMSLayer so something else is 
> going on, but I don't have time right now to trace it down.
> 
> The path to the executables is:   ../lib/ext/etc/  for mrsidgeodecode.exe and 
> mrsidgeoinfo.exe, and also a tmp folder for the tiles.
> 
> regards,
> 
> Larry
> 
> On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud  > wrote:
> 
> Hi Stefan, Larry
> 
> While updating WMS code, I've found several patches from Stefan
> about SID Layer handling.
> Seems that SIDLayer inherits from WMS Layer, hence, patches
> have been added in WMS code to exclude cases where WMS Layers
> are instances of SIDLayer.
> 
> The design seems error prone and probably currently buggy (and
> maybe more now that I've updated the code with WMS only in
> mind).
> Maybe the model would need an abstract parent for both WMS
> and SID Layer.
> The reason of sharing is probably the common createImage()
> method, but WMSLayer has also references to the WMService
> which have nothing to do with SID Layer (AFAIK).
> 
> Agree with that ? Any comment ?
> 
> Regards,
> 
> Michaël
> 
> 
> --
> Precog is a next-generation analytics platform capable of advanced
> analytics on semi-structured data. The platform includes APIs for building
> apps and a phenomenal toolset for data science. Developers can use
> our toolset for easy data analysis & visualization. Get a free account!
> http://www2.precog.com/precogplatform/slashdotnewsletter
> ___
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net 
> 
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> 
> 
> 
> 
> --
> Precog is a next-generation analytics platform capable of advanced
> analytics on semi-structured data. The platform includes APIs for building
> apps and a phenomenal toolset for data science. Developers can use
> our toolset for easy data analysis & visualization. Get a free account!
> http://www2.precog.com/precogplatform/slashdotnewsletter
> 
> 
> 
> ___
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> 

--
Precog is a next-generation analytics platform capable of advanced
analytics on semi-structured data. The platform includes APIs for building
apps and a phenomenal toolset for data science. Developers can use
our toolset for easy data analysis & visualization. Get a free account!
http://www2.precog.com/precogplatform/slashdotnewsletter
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread Larry Becker
OK, I've located the problem that was causing the NPE.  The MRSID
executables now depend on the geos.dll (C++ port of the Java Topology
Suite) which must be located in the same directory.  MRSID files now work
fine, although not as well as in SkyJUMP. ;-)  (SkyJUMP has added a simple
plugin called EditMrSID which allows adjustment of dynamic range and
transparancy.)  I can't see a reason to keep the SIDLayer class.  Does
anyone else?

Larry



On Mon, Apr 15, 2013 at 10:10 AM,  wrote:

> is MrSid a format still in use? should we fix it? .. ede
>
> On 15.04.2013 16:46, Larry Becker wrote:
> > HI Michaël,
> >
> >   Yes, you are correct.  I believe his is not the first time changes to
> WMSLayer have affected SIDLayer.  The original SIDLayer was a quick hack
> that never got the refactoring it deserved.  A common abstract parent is
> probably the best practice.
> >
> >   However, I'm not sure why OJ still has the SIDLayer class since it was
> incorporated into the last version of vividsolutions JUMP as
> MrSIDReferencedImage (which is not descended from WMSLayer)..  Is there
> some feature of the original class that isn't in it? The java2xml perhaps
> doesn't support saving to the project as well?   I would like to test this,
> but after an hour of trying I was unable to get a MRSID file to load in OJ.
>  There are many different wiki pages and readme files that contradict each
> other, but I couldn't get any of them to work.  I finally resorted to debug
> and found the required path for the support executables, however I got an
> NPE when I tried to open a SID file  at
> com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
>  This is the version that doesn't depend on WMSLayer so something else is
> going on, but I don't have time right now to trace it down.
> >
> > The path to the executables is:   ../lib/ext/etc/  for
> mrsidgeodecode.exe and mrsidgeoinfo.exe, and also a tmp folder for the
> tiles.
> >
> > regards,
> >
> > Larry
> >
> > On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud <
> michael.mich...@free.fr > wrote:
> >
> > Hi Stefan, Larry
> >
> > While updating WMS code, I've found several patches from Stefan
> > about SID Layer handling.
> > Seems that SIDLayer inherits from WMS Layer, hence, patches
> > have been added in WMS code to exclude cases where WMS Layers
> > are instances of SIDLayer.
> >
> > The design seems error prone and probably currently buggy (and
> > maybe more now that I've updated the code with WMS only in
> > mind).
> > Maybe the model would need an abstract parent for both WMS
> > and SID Layer.
> > The reason of sharing is probably the common createImage()
> > method, but WMSLayer has also references to the WMService
> > which have nothing to do with SID Layer (AFAIK).
> >
> > Agree with that ? Any comment ?
> >
> > Regards,
> >
> > Michaël
> >
> >
> --
> > Precog is a next-generation analytics platform capable of advanced
> > analytics on semi-structured data. The platform includes APIs for
> building
> > apps and a phenomenal toolset for data science. Developers can use
> > our toolset for easy data analysis & visualization. Get a free
> account!
> > http://www2.precog.com/precogplatform/slashdotnewsletter
> > ___
> > Jump-pilot-devel mailing list
> > Jump-pilot-devel@lists.sourceforge.net  Jump-pilot-devel@lists.sourceforge.net>
> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> >
> >
> >
> >
> >
> --
> > Precog is a next-generation analytics platform capable of advanced
> > analytics on semi-structured data. The platform includes APIs for
> building
> > apps and a phenomenal toolset for data science. Developers can use
> > our toolset for easy data analysis & visualization. Get a free account!
> > http://www2.precog.com/precogplatform/slashdotnewsletter
> >
> >
> >
> > ___
> > Jump-pilot-devel mailing list
> > Jump-pilot-devel@lists.sourceforge.net
> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> >
>
>
> --
> Precog is a next-generation analytics platform capable of advanced
> analytics on semi-structured data. The platform includes APIs for building
> apps and a phenomenal toolset for data science. Developers can use
> our toolset for easy data analysis & visualization. Get a free account!
> http://www2.precog.com/precogplatform/slashdotnewsletter
> ___
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>

Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread edgar . soldin
what exactly do you suggest? complete removal, copy&paste ";)" sky jump's 
solution?.. ede

On 15.04.2013 21:20, Larry Becker wrote:
> OK, I've located the problem that was causing the NPE.  The MRSID executables 
> now depend on the geos.dll (C++ port of the Java Topology Suite) which must 
> be located in the same directory.  MRSID files now work fine, although not as 
> well as in SkyJUMP. ;-)  (SkyJUMP has added a simple plugin called EditMrSID 
> which allows adjustment of dynamic range and transparancy.)  I can't see a 
> reason to keep the SIDLayer class.  Does anyone else?
> 
> Larry
> 
> 
> 
> On Mon, Apr 15, 2013 at 10:10 AM,  > wrote:
> 
> is MrSid a format still in use? should we fix it? .. ede
> 
> On 15.04.2013 16:46, Larry Becker wrote:
> > HI Michaël,
> >
> >   Yes, you are correct.  I believe his is not the first time changes to 
> WMSLayer have affected SIDLayer.  The original SIDLayer was a quick hack that 
> never got the refactoring it deserved.  A common abstract parent is probably 
> the best practice.
> >
> >   However, I'm not sure why OJ still has the SIDLayer class since it 
> was incorporated into the last version of vividsolutions JUMP as 
> MrSIDReferencedImage (which is not descended from WMSLayer)..  Is there some 
> feature of the original class that isn't in it? The java2xml perhaps doesn't 
> support saving to the project as well?   I would like to test this, but after 
> an hour of trying I was unable to get a MRSID file to load in OJ.  There are 
> many different wiki pages and readme files that contradict each other, but I 
> couldn't get any of them to work.  I finally resorted to debug and found the 
> required path for the support executables, however I got an NPE when I tried 
> to open a SID file  at 
> com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
>   This is the version that doesn't depend on WMSLayer so something else is 
> going on, but I don't have time right now to trace it down.
> >
> > The path to the executables is:   ../lib/ext/etc/  for 
> mrsidgeodecode.exe and mrsidgeoinfo.exe, and also a tmp folder for the tiles.
> >
> > regards,
> >
> > Larry
> >
> > On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud 
> mailto:michael.mich...@free.fr> 
> >> wrote:
> >
> > Hi Stefan, Larry
> >
> > While updating WMS code, I've found several patches from Stefan
> > about SID Layer handling.
> > Seems that SIDLayer inherits from WMS Layer, hence, patches
> > have been added in WMS code to exclude cases where WMS Layers
> > are instances of SIDLayer.
> >
> > The design seems error prone and probably currently buggy (and
> > maybe more now that I've updated the code with WMS only in
> > mind).
> > Maybe the model would need an abstract parent for both WMS
> > and SID Layer.
> > The reason of sharing is probably the common createImage()
> > method, but WMSLayer has also references to the WMService
> > which have nothing to do with SID Layer (AFAIK).
> >
> > Agree with that ? Any comment ?
> >
> > Regards,
> >
> > Michaël
> >
> > 
> --
> > Precog is a next-generation analytics platform capable of advanced
> > analytics on semi-structured data. The platform includes APIs for 
> building
> > apps and a phenomenal toolset for data science. Developers can use
> > our toolset for easy data analysis & visualization. Get a free 
> account!
> > http://www2.precog.com/precogplatform/slashdotnewsletter
> > ___
> > Jump-pilot-devel mailing list
> > Jump-pilot-devel@lists.sourceforge.net 
>  
>  >
> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
> >
> >
> >
> >
> > 
> --
> > Precog is a next-generation analytics platform capable of advanced
> > analytics on semi-structured data. The platform includes APIs for 
> building
> > apps and a phenomenal toolset for data science. Developers can use
> > our toolset for easy data analysis & visualization. Get a free account!
> > http://www2.precog.com/precogplatform/slashdotnewsletter
> >
> >
> >
> > ___
> > Jump-pilot-devel mailing list
> > Jump-pilot-devel@lists.sourceforge.net 
> 
> >

Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread Michaël Michaud

Hi Larry,
OK, I've located the problem that was causing the NPE.  The MRSID 
executables now depend on the geos.dll (C++ port of the Java Topology 
Suite) which must be located in the same directory.  MRSID files now 
work fine, although not as well as in SkyJUMP. ;-)  (SkyJUMP has added 
a simple plugin called EditMrSID which allows adjustment of dynamic 
range and transparancy.)  I can't see a reason to keep the SIDLayer 
class.  Does anyone else?

Thanks a lot,

I hadn't even caught that SIDLayer was deprecated in favour of 
MrSIDReferencedImage.
Probably the reason why Stefan removed some references to SID Layers in 
WMSLayer code.


SIDLayer has probably to be removed then.

On Mon, Apr 15, 2013 at 10:10 AM, > wrote:


is MrSid a format still in use? should we fix it? .. ede

Don't now. I personnaly do not wish to spend time to support this 
format, but if we (Larry) can
keep it working without too much effort, maybe some users will be happy 
to have it.


Michaël



On 15.04.2013 16:46, Larry Becker wrote:
> HI Michaël,
>
>   Yes, you are correct.  I believe his is not the first time
changes to WMSLayer have affected SIDLayer.  The original SIDLayer
was a quick hack that never got the refactoring it deserved.  A
common abstract parent is probably the best practice.
>
>   However, I'm not sure why OJ still has the SIDLayer class
since it was incorporated into the last version of vividsolutions
JUMP as MrSIDReferencedImage (which is not descended from
WMSLayer)..  Is there some feature of the original class that
isn't in it? The java2xml perhaps doesn't support saving to the
project as well?   I would like to test this, but after an hour of
trying I was unable to get a MRSID file to load in OJ.  There are
many different wiki pages and readme files that contradict each
other, but I couldn't get any of them to work.  I finally resorted
to debug and found the required path for the support executables,
however I got an NPE when I tried to open a SID file  at

com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
 This is the version that doesn't depend on WMSLayer so something
else is going on, but I don't have time right now to trace it down.
>
> The path to the executables is:   ../lib/ext/etc/  for
mrsidgeodecode.exe and mrsidgeoinfo.exe, and also a tmp folder for
the tiles.
>
> regards,
>
> Larry
>
> On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud
mailto:michael.mich...@free.fr>
>>
wrote:
>
> Hi Stefan, Larry
>
> While updating WMS code, I've found several patches from Stefan
> about SID Layer handling.
> Seems that SIDLayer inherits from WMS Layer, hence, patches
> have been added in WMS code to exclude cases where WMS Layers
> are instances of SIDLayer.
>
> The design seems error prone and probably currently buggy (and
> maybe more now that I've updated the code with WMS only in
> mind).
> Maybe the model would need an abstract parent for both WMS
> and SID Layer.
> The reason of sharing is probably the common createImage()
> method, but WMSLayer has also references to the WMService
> which have nothing to do with SID Layer (AFAIK).
>
> Agree with that ? Any comment ?
>
> Regards,
>
> Michaël
>
>

--
> Precog is a next-generation analytics platform capable of
advanced
> analytics on semi-structured data. The platform includes
APIs for building
> apps and a phenomenal toolset for data science. Developers
can use
> our toolset for easy data analysis & visualization. Get a
free account!
> http://www2.precog.com/precogplatform/slashdotnewsletter
> ___
> Jump-pilot-devel mailing list
> Jump-pilot-devel@lists.sourceforge.net

>
> https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>
>
>
>
>

--
> Precog is a next-generation analytics platform capable of advanced
> analytics on semi-structured data. The platform includes APIs
for building
> apps and a phenomenal toolset for data science. Developers can use
> our toolset for easy data analysis & visualization. Get a free
account!
> http://www2.precog.com/precogplatform/slashdotnewsletter
>
>
>
> _

Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread Larry Becker
The main task to keep it working is maintaining the command line parameters
to support the latest version of the MRSIDS utilities.  I'll be doing that
for SkyJUMP so it is a simple matter to keep OJ up to date too.  MRSID,
ECW,  and ArcSDE are OpenJump's gateway to the non-free GIS world.  Some of
us are still tied to the commercial and government sector.

regards,

Larry

On Mon, Apr 15, 2013 at 2:47 PM, Michaël Michaud wrote:

>  Hi Larry,
>
> OK, I've located the problem that was causing the NPE.  The MRSID
> executables now depend on the geos.dll (C++ port of the Java Topology
> Suite) which must be located in the same directory.  MRSID files now work
> fine, although not as well as in SkyJUMP. ;-)  (SkyJUMP has added a simple
> plugin called EditMrSID which allows adjustment of dynamic range and
> transparancy.)  I can't see a reason to keep the SIDLayer class.  Does
> anyone else?
>
> Thanks a lot,
>
> I hadn't even caught that SIDLayer was deprecated in favour of
> MrSIDReferencedImage.
> Probably the reason why Stefan removed some references to SID Layers in
> WMSLayer code.
>
> SIDLayer has probably to be removed then.
>
> On Mon, Apr 15, 2013 at 10:10 AM,  wrote:
>
>> is MrSid a format still in use? should we fix it? .. ede
>>
>  Don't now. I personnaly do not wish to spend time to support this
> format, but if we (Larry) can
> keep it working without too much effort, maybe some users will be happy to
> have it.
>
> Michaël
>
>
>> On 15.04.2013 16:46, Larry Becker wrote:
>> > HI Michaël,
>> >
>> >   Yes, you are correct.  I believe his is not the first time changes to
>> WMSLayer have affected SIDLayer.  The original SIDLayer was a quick hack
>> that never got the refactoring it deserved.  A common abstract parent is
>> probably the best practice.
>> >
>> >   However, I'm not sure why OJ still has the SIDLayer class since it
>> was incorporated into the last version of vividsolutions JUMP as
>> MrSIDReferencedImage (which is not descended from WMSLayer)..  Is there
>> some feature of the original class that isn't in it? The java2xml perhaps
>> doesn't support saving to the project as well?   I would like to test this,
>> but after an hour of trying I was unable to get a MRSID file to load in OJ.
>>  There are many different wiki pages and readme files that contradict each
>> other, but I couldn't get any of them to work.  I finally resorted to debug
>> and found the required path for the support executables, however I got an
>> NPE when I tried to open a SID file  at
>> com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
>>  This is the version that doesn't depend on WMSLayer so something else is
>> going on, but I don't have time right now to trace it down.
>> >
>> > The path to the executables is:   ../lib/ext/etc/  for
>> mrsidgeodecode.exe and mrsidgeoinfo.exe, and also a tmp folder for the
>> tiles.
>> >
>> > regards,
>> >
>> > Larry
>> >
>>  > On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud <
>> michael.mich...@free.fr > wrote:
>> >
>> > Hi Stefan, Larry
>> >
>> > While updating WMS code, I've found several patches from Stefan
>> > about SID Layer handling.
>> > Seems that SIDLayer inherits from WMS Layer, hence, patches
>> > have been added in WMS code to exclude cases where WMS Layers
>> > are instances of SIDLayer.
>> >
>> > The design seems error prone and probably currently buggy (and
>> > maybe more now that I've updated the code with WMS only in
>> > mind).
>> > Maybe the model would need an abstract parent for both WMS
>> > and SID Layer.
>> > The reason of sharing is probably the common createImage()
>> > method, but WMSLayer has also references to the WMService
>> > which have nothing to do with SID Layer (AFAIK).
>> >
>> > Agree with that ? Any comment ?
>> >
>> > Regards,
>> >
>> > Michaël
>> >
>> >
>> --
>> > Precog is a next-generation analytics platform capable of advanced
>> > analytics on semi-structured data. The platform includes APIs for
>> building
>> > apps and a phenomenal toolset for data science. Developers can use
>> > our toolset for easy data analysis & visualization. Get a free
>> account!
>> > http://www2.precog.com/precogplatform/slashdotnewsletter
>> > ___
>> > Jump-pilot-devel mailing list
>>  > Jump-pilot-devel@lists.sourceforge.net > Jump-pilot-devel@lists.sourceforge.net>
>> > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
>>  >
>> >
>> >
>> >
>> >
>> --
>> > Precog is a next-generation analytics platform capable of advanced
>> > analytics on semi-structured data. The platform includes APIs for
>> building
>> > apps and a phenomenal toolset for data scie

Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-15 Thread Michaël Michaud

Hi,

For info (wiki will probably need an update about this format)
The exe is available on http://www.lizardtech.com/
under category
Downloads > Tools & Utilities > GeoExpress Command Line Utilities
Don't know if libroject directory is needed and where it must be installed

First time I tried it, I get an error message and had to change my windows
regional parameter from , (comma) to . (dot).

I still get no image, but I have not yet followed the whole procedure 
described in

https://sourceforge.net/apps/mediawiki/jump-pilot/index.php?title=MrSID
Do not have error message anymore...

to follow...

Michaël

PS : I just removed the SIDLayer and AddSIDLayerPlugIn class from the 
project.


The main task to keep it working is maintaining the command line 
parameters to support the latest version of the MRSIDS utilities.  
I'll be doing that for SkyJUMP so it is a simple matter to keep OJ up 
to date too.  MRSID,  ECW,  and ArcSDE are OpenJump's gateway to the 
non-free GIS world.  Some of us are still tied to the commercial and 
government sector.


regards,

Larry

On Mon, Apr 15, 2013 at 2:47 PM, Michaël Michaud 
mailto:michael.mich...@free.fr>> wrote:


Hi Larry,

OK, I've located the problem that was causing the NPE.  The MRSID
executables now depend on the geos.dll (C++ port of the Java
Topology Suite) which must be located in the same directory. 
MRSID files now work fine, although not as well as in SkyJUMP.

;-)  (SkyJUMP has added a simple plugin called EditMrSID which
allows adjustment of dynamic range and transparancy.)  I can't
see a reason to keep the SIDLayer class.  Does anyone else?

Thanks a lot,

I hadn't even caught that SIDLayer was deprecated in favour of
MrSIDReferencedImage.
Probably the reason why Stefan removed some references to SID
Layers in WMSLayer code.

SIDLayer has probably to be removed then.


On Mon, Apr 15, 2013 at 10:10 AM, mailto:edgar.sol...@web.de>> wrote:

is MrSid a format still in use? should we fix it? .. ede


Don't now. I personnaly do not wish to spend time to support this
format, but if we (Larry) can
keep it working without too much effort, maybe some users will be
happy to have it.

Michaël



On 15.04.2013 16:46, Larry Becker wrote:
> HI Michaël,
>
>   Yes, you are correct.  I believe his is not the first
time changes to WMSLayer have affected SIDLayer.  The
original SIDLayer was a quick hack that never got the
refactoring it deserved.  A common abstract parent is
probably the best practice.
>
>   However, I'm not sure why OJ still has the SIDLayer class
since it was incorporated into the last version of
vividsolutions JUMP as MrSIDReferencedImage (which is not
descended from WMSLayer)..  Is there some feature of the
original class that isn't in it? The java2xml perhaps doesn't
support saving to the project as well?   I would like to test
this, but after an hour of trying I was unable to get a MRSID
file to load in OJ.  There are many different wiki pages and
readme files that contradict each other, but I couldn't get
any of them to work.  I finally resorted to debug and found
the required path for the support executables, however I got
an NPE when I tried to open a SID file  at

com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
 This is the version that doesn't depend on WMSLayer so
something else is going on, but I don't have time right now
to trace it down.
>
> The path to the executables is: ../lib/ext/etc/  for
mrsidgeodecode.exe and mrsidgeoinfo.exe, and also a tmp
folder for the tiles.
>
> regards,
>
> Larry
>
> On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud
mailto:michael.mich...@free.fr>
>> wrote:
>
> Hi Stefan, Larry
>
> While updating WMS code, I've found several patches
from Stefan
> about SID Layer handling.
> Seems that SIDLayer inherits from WMS Layer, hence, patches
> have been added in WMS code to exclude cases where WMS
Layers
> are instances of SIDLayer.
>
> The design seems error prone and probably currently
buggy (and
> maybe more now that I've updated the code with WMS only in
> mind).
> Maybe the model would need an abstract parent for both WMS
> and SID Layer.
> The reason of sharing is probably the common createImage()
> method, but WMSLayer has also references to the WMService
> which have nothing to do with S

Re: [JPP-Devel] WMS Layer / SID Layer

2013-04-16 Thread edgar . soldin
Michael,

while you're at it. could you please move the mrsid binary path to 'lib/native 
? it is already in the PATH variable. all you have to do is 

- remove these senseless absolute paths for the binaries in MrSIDImageFactory 
(ca. line 100)
- assign and use the system temp folder
- move the binaries into 'lib/native'

if MrSID is not working properly you might want to compare skyjump parameter in 
MrSIDReferencedImage (ca. line 233) with those used in SkyJUMP.

it might be a good idea to hint the date and version of the binaries in the 
wiki once it is working again.

..ede

On 16.04.2013 08:38, Michaël Michaud wrote:
> Hi,
> 
> For info (wiki will probably need an update about this format)
> The exe is available on http://www.lizardtech.com/
> under category
> Downloads > Tools & Utilities > GeoExpress Command Line Utilities
> Don't know if libroject directory is needed and where it must be installed
> 
> First time I tried it, I get an error message and had to change my windows
> regional parameter from , (comma) to . (dot).
> 
> I still get no image, but I have not yet followed the whole procedure 
> described in
> https://sourceforge.net/apps/mediawiki/jump-pilot/index.php?title=MrSID
> Do not have error message anymore...
> 
> to follow...
> 
> Michaël
> 
> PS : I just removed the SIDLayer and AddSIDLayerPlugIn class from the project.
> 
>> The main task to keep it working is maintaining the command line parameters 
>> to support the latest version of the MRSIDS utilities.  I'll be doing that 
>> for SkyJUMP so it is a simple matter to keep OJ up to date too.  MRSID,  
>> ECW,  and ArcSDE are OpenJump's gateway to the non-free GIS world.  Some of 
>> us are still tied to the commercial and government sector.
>>
>> regards,
>>
>> Larry
>>
>> On Mon, Apr 15, 2013 at 2:47 PM, Michaël Michaud > > wrote:
>>
>> Hi Larry,
>>> OK, I've located the problem that was causing the NPE.  The MRSID 
>>> executables now depend on the geos.dll (C++ port of the Java Topology 
>>> Suite) which must be located in the same directory.  MRSID files now work 
>>> fine, although not as well as in SkyJUMP. ;-)  (SkyJUMP has added a simple 
>>> plugin called EditMrSID which allows adjustment of dynamic range and 
>>> transparancy.)  I can't see a reason to keep the SIDLayer class.  Does 
>>> anyone else?
>> Thanks a lot,
>>
>> I hadn't even caught that SIDLayer was deprecated in favour of 
>> MrSIDReferencedImage.
>> Probably the reason why Stefan removed some references to SID Layers in 
>> WMSLayer code.
>>
>> SIDLayer has probably to be removed then.
>>
>>> On Mon, Apr 15, 2013 at 10:10 AM, >> > wrote:
>>>
>>> is MrSid a format still in use? should we fix it? .. ede
>>>
>> Don't now. I personnaly do not wish to spend time to support this 
>> format, but if we (Larry) can
>> keep it working without too much effort, maybe some users will be happy 
>> to have it.
>>
>> Michaël
>>
>>>
>>> On 15.04.2013 16:46, Larry Becker wrote:
>>> > HI Michaël,
>>> >
>>> >   Yes, you are correct.  I believe his is not the first time 
>>> changes to WMSLayer have affected SIDLayer.  The original SIDLayer was a 
>>> quick hack that never got the refactoring it deserved.  A common abstract 
>>> parent is probably the best practice.
>>> >
>>> >   However, I'm not sure why OJ still has the SIDLayer class since 
>>> it was incorporated into the last version of vividsolutions JUMP as 
>>> MrSIDReferencedImage (which is not descended from WMSLayer)..  Is there 
>>> some feature of the original class that isn't in it? The java2xml perhaps 
>>> doesn't support saving to the project as well?   I would like to test this, 
>>> but after an hour of trying I was unable to get a MRSID file to load in OJ. 
>>>  There are many different wiki pages and readme files that contradict each 
>>> other, but I couldn't get any of them to work.  I finally resorted to debug 
>>> and found the required path for the support executables, however I got an 
>>> NPE when I tried to open a SID file  at 
>>> com.vividsolutions.jump.workbench.imagery.mrsid.MrSIDReferencedImage.getEnvelope(MrSIDReferencedImage.java:98).
>>>   This is the version that doesn't depend on WMSLayer so something else is 
>>> going on, but I don't have time right now to trace it down.
>>> >
>>> > The path to the executables is:   ../lib/ext/etc/  for 
>>> mrsidgeodecode.exe and mrsidgeoinfo.exe, and also a tmp folder for the 
>>> tiles.
>>> >
>>> > regards,
>>> >
>>> > Larry
>>> >
>>> > On Sun, Apr 14, 2013 at 11:34 AM, Michaël Michaud 
>>> mailto:michael.mich...@free.fr> 
>>> >> wrote:
>>> >
>>> > Hi Stefan, Larry
>>> >
>>> > While updating WMS code, I've found several patches fro