[GRASS-dev] [GRASS GIS] #3142: Implementing SLIC image segmentation

[GRASS GIS]

#3142: Implementing SLIC image segmentation
---------------------------+-------------------------------------
  Reporter:  SCrommelinck  |      Owner:  grass-dev@…
      Type:  enhancement   |     Status:  new
  Priority:  normal        |  Milestone:  7.2.1
 Component:  Imagery       |    Version:  unspecified
Resolution:                |   Keywords:  SLIC image segmentation
       CPU:  Unspecified   |   Platform:  Unspecified
---------------------------+-------------------------------------

Comment (by mlennert):

 Replying to [comment:18 rashadkm]:
 > here is first version on grass7 addons:
 > https://trac.osgeo.org/grass/browser/grass-
 addons/grass7/imagery/i.superpixels.slic
 >
 > can we close this ticket?

 There are still one [https://lists.osgeo.org/pipermail/grass-
 dev/2017-January/083745.html comment from Markus] that you didn't apply:
 the memory allocation for distvec. Just uncomment
 [https://trac.osgeo.org/grass/browser/grass-
 addons/grass7/imagery/i.superpixels.slic/main.c#L353 lines 353 and
 following], not forgetting to add the G_free() call later (see the patch
 in #3247). Otherwise the memory is allocated in the stack, where there
 might not be enough available, rather than the heap.

 When this is done, I agree that this ticket can be closed and we can open
 new tickets for any other problems.

 However, while you're at it: could you also consider the two additions
 proposed in that same patch of #3247, assigning a color table and adding
 the module call to the raster map's history ?

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3142#comment:19>
GRASS GIS <https://grass.osgeo.org>