GeoTools
  1. GeoTools
  2. GEOT-3336

Raster to vector process to use jai-tools operator

    Details

    • Type: Improvement Improvement
    • Status: Open Open
    • Priority: Minor Minor
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: 2.7.6
    • Component/s: None
    • Labels:
      None

      Description

      The unsupported process module includes RasterToVectorProcess. I'm transferring the algorithm code from this class into a new "Vectorize" JAI operator for the jai-tools project. This work is unearthing a few bugs as well as forcing me to make the code more understandable and modular.

      When done, the RasterToVectorProcess class will become a geospatial wrapper for the jai-tools operator.
      Alternatively, it could be moved in with the other GeoTools coverage operators (but I still don't quite understand how they all fit together).

      At this stage I'm only proposing to make this change on trunk, but I'd be happy to back-port it to 2.6.x if any of the bug fixes and tweaks are important to users.

        Activity

        Hide
        Michael Bedward added a comment -
        Simone - we briefly discussed this suggestion a while ago and you mentioned that the vectorizing code could be rescued from the process module and put into coverage.

        Re-assigning the issue to you. Please feel free to proceed with it or close it as you wish.
        Show
        Michael Bedward added a comment - Simone - we briefly discussed this suggestion a while ago and you mentioned that the vectorizing code could be rescued from the process module and put into coverage. Re-assigning the issue to you. Please feel free to proceed with it or close it as you wish.
        Hide
        Michael Bedward added a comment -
        From Andrea on the dev list: "GeoServer has the processes wrapping the jaitools operators, so I guess we just need to grab those"
        Show
        Michael Bedward added a comment - From Andrea on the dev list: "GeoServer has the processes wrapping the jaitools operators, so I guess we just need to grab those"

          People

          • Assignee:
            Simone Giannecchini
            Reporter:
            Michael Bedward
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: