Opened 9 months ago

Closed 9 months ago

Last modified 9 months ago

#3932 closed defect (fixed)

i.vi: NDWI unnecessarily requires red band

Reported by: sbl Owned by: grass-dev@…
Priority: normal Milestone: 7.8.1
Component: Imagery Version: git-releasebranch78
Keywords: i.vi Cc:
CPU: All Platform: All

Description

i.vi requires a red band in any case, even if e.g. NDWI does not need one. So, in order to make the module run, a map has to be given also for the red band. That seems like unexpected behavior to me. Maybe better to leave all input bands optional and add a check for the requested VI if required bands are provided...

Change History (4)

comment:2 Changed 9 months ago by neteler

Related: also the i.wi addon should be likewise updated:

https://github.com/OSGeo/grass-addons/blob/master/grass7/imagery/i.wi/main.c

comment:4 Changed 9 months ago by sbl

Personally, I would say, ideally, i.vi and i.wi would be merged into one module e.g. i.spectral.index, where computation of all sorts of spectral indices is collected...

That would be most convenient for users and it would reduce maintenance burden (though the modules would have to be merged first of course). Worth a new ticket?

Note: See TracTickets for help on using tickets.