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, <edgar.sol...@web.de> 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 <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 <mailto:
> 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
>
------------------------------------------------------------------------------
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

Reply via email to