[GRASS-dev] [GRASS GIS] #3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI

GDAL · October 19, 2016, 11:41am

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------------+-------------------------
Reporter: mlennert | Owner: grass-dev@…
Type: enhancement | Status: new
Priority: normal | Milestone:
Component: Imagery | Version: svn-trunk
Keywords: i.in.spotvgt bands | CPU: Unspecified
Platform: Unspecified |
--------------------------------+-------------------------
Currently, i.in.spotvgt only imports NDVI bands. The attached patch
againts trunk allows importing any band.

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3186>
GRASS GIS <https://grass.osgeo.org>

GDAL · October 19, 2016, 12:27pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.6
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------
Changes (by mlennert):

* milestone: => 7.0.6

Comment:

Am actually working on a different version that loops over all bands, so
please ignore the current patch for now...

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

GDAL · October 19, 2016, 12:36pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.6
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------
Changes (by mlennert):

* Attachment "i_in_spotvgt_all_bands.diff" added.

patch to allow import of any band

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3186>
GRASS GIS <https://grass.osgeo.org>

GDAL · October 19, 2016, 12:36pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.6
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------
Changes (by mlennert):

* Attachment "i_in_spotvgt_all_bands.diff" removed.

patch to allow import of any band

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3186>
GRASS GIS <https://grass.osgeo.org>

GDAL · October 19, 2016, 12:43pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.6
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------

Comment (by mlennert):

I've replaced the patch with a new one, that loops through all bands. By
default it only imports NDVI (as before), but by setting the -b flag the
use can request import of all bands (B0, B2, B3, MIR, and NDVI).

This changes the module API a bit as now the user points the module to the
metadata file (0001_LOG.TXT) and not to the NDVI file. I would be possible
to rewrite it so that the user can point to either of the two. Or just the
NDVI file... I'll get back to it when I find the time, unless someone else
wants to.

As is, the patch does work for looping, though.

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

GDAL · March 2, 2018, 3:23pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.7
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------
Changes (by mlennert):

* Attachment "i_in_spotvgt_any_band.diff" added.

patch that allows importing any band (one by one) - provided by Tudor-Emil
Coman

--
Ticket URL: <https://trac.osgeo.org/grass/ticket/3186>
GRASS GIS <https://grass.osgeo.org>

GDAL · March 2, 2018, 3:30pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.7
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------

Comment (by mlennert):

I've just attached the patch provided by Tudor. It allows importing any
band, one by one. Probably more in line with the KISS principle than my
patch trying to loop over all files.

Any opinions from SPOT users out there ?

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

GDAL · April 28, 2019, 2:32pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.0.7
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------

Comment (by martinl):

Still relevant?

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

GDAL · May 6, 2019, 2:53pm

#3186: [PATCH] i.in.spotvgt: allow import of any band, not only NDVI
--------------------------+--------------------------------
  Reporter: mlennert | Owner: grass-dev@…
      Type: enhancement | Status: new
  Priority: normal | Milestone: 7.8.0
Component: Imagery | Version: svn-trunk
Resolution: | Keywords: i.in.spotvgt bands
       CPU: Unspecified | Platform: Unspecified
--------------------------+--------------------------------
Changes (by mlennert):

* milestone: 7.0.7 => 7.8.0

Comment:

I think this is still relevant, but am not a regular SPOT imagery user, so
am not 100% sure the patch is acceptable to all. Best bet is probably to
just apply it and see

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