Re: [JPP-Devel] WMS Layer / SID Layer
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 michael.mich...@free.fr 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, edgar.sol...@web.de 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 michael.mich...@free.fr mailto:michael.mich...@free.fr mailto: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
Re: [JPP-Devel] WMS Layer / SID Layer
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 michael.mich...@free.fr 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, edgar.sol...@web.de 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 michael.mich...@free.fr mailto:michael.mich...@free.fr mailto: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
Re: [JPP-Devel] WMS Layer / SID Layer
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.frwrote: 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
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
Re: [JPP-Devel] WMS Layer / SID Layer
what exactly do you suggest? complete removal, copypaste ;) 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, edgar.sol...@web.de mailto: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 mailto: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 mailto: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 mailto:Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
Re: [JPP-Devel] WMS Layer / SID Layer
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, edgar.sol...@web.de 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 michael.mich...@free.fr mailto:michael.mich...@free.fr mailto: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 mailto: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
Re: [JPP-Devel] WMS Layer / SID Layer
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 michael.mich...@free.frwrote: 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, 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 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
[JPP-Devel] WMS Layer / SID Layer
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