Opened 8 years ago
Last modified 5 years ago
#3186 new enhancement
[PATCH] i.in.spotvgt: allow import of any band, not only NDVI
Reported by: | mlennert | Owned by: | |
---|---|---|---|
Priority: | normal | Milestone: | 7.8.3 |
Component: | Imagery | Version: | svn-trunk |
Keywords: | i.in.spotvgt bands | Cc: | |
CPU: | Unspecified | Platform: | Unspecified |
Description
Currently, i.in.spotvgt only imports NDVI bands. The attached patch againts trunk allows importing any band.
Attachments (2)
Change History (12)
comment:1 by , 8 years ago
Milestone: | → 7.0.6 |
---|
comment:2 by , 8 years ago
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.
comment:3 by , 7 years ago
Milestone: | 7.0.6 → 7.0.7 |
---|
by , 7 years ago
Attachment: | i_in_spotvgt_any_band.diff added |
---|
patch that allows importing any band (one by one) - provided by Tudor-Emil Coman
comment:4 by , 7 years ago
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 ?
comment:6 by , 6 years ago
Milestone: | 7.0.7 → 7.8.0 |
---|
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 ;-)
comment:10 by , 5 years ago
Milestone: | → 7.8.3 |
---|
Am actually working on a different version that loops over all bands, so please ignore the current patch for now...