GeoTools

deprecate inner coverage's ImageLayout2 class in favor of jaitools's class

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: 8.0-M0
  • Fix Version/s: 8.0-M1
  • Component/s: coverage
  • Labels:
    None

Description

The Coverage module defines a org.geotools.image.ImageLayout2
The jai-tools jt-utils module defines a jaitools.imageutils.ImageLayout2 class which has the same implementation.

Since coverage now depends on jaitools utils, we could remove the coverage's class in favor of the jai-tools one.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Michael Bedward added a comment -
Daniele,

It is lucky you posted this issue because I was thinking to remove the ImageLayout2 class from the next JAITools release and dump it into a branch with other code that Simone committed a long time ago (combine, hysteresis and nodatafilter operators).

The class looks like rough code and has no proper documentation. I will be releasing version 1.2 soon so if you want to keep ImageLayout2 in JAITools please tidy it up and document it in the 1.2.x branch as soon as possible. It will need to be swapped to the BSD-2 licence as well.

Version 1.2 of JAITools will use org.jaitools as the groupId and root package name.

cheers
Michael

Show
Michael Bedward added a comment - Daniele, It is lucky you posted this issue because I was thinking to remove the ImageLayout2 class from the next JAITools release and dump it into a branch with other code that Simone committed a long time ago (combine, hysteresis and nodatafilter operators). The class looks like rough code and has no proper documentation. I will be releasing version 1.2 soon so if you want to keep ImageLayout2 in JAITools please tidy it up and document it in the 1.2.x branch as soon as possible. It will need to be swapped to the BSD-2 licence as well. Version 1.2 of JAITools will use org.jaitools as the groupId and root package name. cheers Michael
Hide
Permalink
Daniele Romagnoli added a comment -
Fixed as revision 37531
Show
Daniele Romagnoli added a comment - Fixed as revision 37531

People

  • Assignee:
    Daniele Romagnoli
    Reporter:
    Daniele Romagnoli
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: