[JPP-Devel] request for commit approval

2009-10-28 Thread Stefan Steiniger
Hei all,

I have started working with Sextante, the raster processing library 
(within my university project). It is possible to use Sextante external 
from OJ (i.e. it comes with its own GUI) and Sextante utilizes the 
new/old Pirol RasterImageLayer class to transfer raster data. However, 
it can also use OJ VectorLayers as input.

Now I wanted a function that creates points/polygons out of an input 
raster, which requires some basic access methods to rasters. As I am a 
lazy (or clever?) person I have copied the Sextante base classes into 
OpenJUMP (and renamed most of them), so that

i) one can perform basic operations on Pirols RasterImageLayers and 
generate new output layers

ii) one (probably) can create portions of new Sextante algorithms 
directly in OpenJUMP and later need only to adapt the class names when 
converting to a Sextante processing algorithm.

possible negative aspects:

iii) doubling of code between OJ and Sextante (not sure, but I hope 
there won't be much updates to the Sextante base classes/interfaces)

iv) I could have tried to use only functions provided with Pirols 
RasterImage class (However, Sextante provides a nice set of convenience 
methods, and seems to be very well designed).

The following classes would be added to the OJ core:

path: org.openjump.core.rasterimage.sextante.*
AbstractSextanteRasterLayer = AbstractRasterLayer
ISextanteDataObject = IDataObject
ISextanteLayer = ILayer
ISextanteRasterLayer = IRasterLayer
OpenJUMPSextanteRasterLayer = OpenJUMPRasterLayer

path: org.openjump.core.rasterimage.sextante.rasterWrappers*
GriddCell
GridExtent
GridWrapper
GridWrapperInterpolated
GridWrapperNotInterpolated

Any comments on this?
Note, I would like to have feedback until the beginning of next week.

stefan

--
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


Re: [JPP-Devel] request for commit approval

2009-10-28 Thread Sunburned Surveyor
Perhaps we should keep these changes in a separate branch of the SVN
and run/try a seperate build for a while?

I would personally like to keep the core as lean as possible. Can we
add the needed classes via a plug-in instead?

SS

On Wed, Oct 28, 2009 at 11:09 AM, Stefan Steiniger sst...@geo.uzh.ch wrote:
 Hei all,

 I have started working with Sextante, the raster processing library
 (within my university project). It is possible to use Sextante external
 from OJ (i.e. it comes with its own GUI) and Sextante utilizes the
 new/old Pirol RasterImageLayer class to transfer raster data. However,
 it can also use OJ VectorLayers as input.

 Now I wanted a function that creates points/polygons out of an input
 raster, which requires some basic access methods to rasters. As I am a
 lazy (or clever?) person I have copied the Sextante base classes into
 OpenJUMP (and renamed most of them), so that

 i) one can perform basic operations on Pirols RasterImageLayers and
 generate new output layers

 ii) one (probably) can create portions of new Sextante algorithms
 directly in OpenJUMP and later need only to adapt the class names when
 converting to a Sextante processing algorithm.

 possible negative aspects:

 iii) doubling of code between OJ and Sextante (not sure, but I hope
 there won't be much updates to the Sextante base classes/interfaces)

 iv) I could have tried to use only functions provided with Pirols
 RasterImage class (However, Sextante provides a nice set of convenience
 methods, and seems to be very well designed).

 The following classes would be added to the OJ core:

 path: org.openjump.core.rasterimage.sextante.*
 AbstractSextanteRasterLayer = AbstractRasterLayer
 ISextanteDataObject = IDataObject
 ISextanteLayer = ILayer
 ISextanteRasterLayer = IRasterLayer
 OpenJUMPSextanteRasterLayer = OpenJUMPRasterLayer

 path: org.openjump.core.rasterimage.sextante.rasterWrappers*
 GriddCell
 GridExtent
 GridWrapper
 GridWrapperInterpolated
 GridWrapperNotInterpolated

 Any comments on this?
 Note, I would like to have feedback until the beginning of next week.

 stefan

 --
 Come build with us! The BlackBerry(R) Developer Conference in SF, CA
 is the only developer event you need to attend this year. Jumpstart your
 developing skills, take BlackBerry mobile applications to market and stay
 ahead of the curve. Join us from November 9 - 12, 2009. Register now!
 http://p.sf.net/sfu/devconference
 ___
 Jump-pilot-devel mailing list
 Jump-pilot-devel@lists.sourceforge.net
 https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel


--
Come build with us! The BlackBerry(R) Developer Conference in SF, CA
is the only developer event you need to attend this year. Jumpstart your
developing skills, take BlackBerry mobile applications to market and stay 
ahead of the curve. Join us from November 9 - 12, 2009. Register now!
http://p.sf.net/sfu/devconference
___
Jump-pilot-devel mailing list
Jump-pilot-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel