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: grass-dev@…
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)

i_in_spotvgt_all_bands.diff (9.3 KB ) - added by mlennert 8 years ago.
patch to allow import of any band
i_in_spotvgt_any_band.diff (8.1 KB ) - added by mlennert 7 years ago.
patch that allows importing any band (one by one) - provided by Tudor-Emil Coman

Download all attachments as: .zip

Change History (12)

comment:1 by mlennert, 8 years ago

Milestone: 7.0.6

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

by mlennert, 8 years ago

Attachment: i_in_spotvgt_all_bands.diff added

patch to allow import of any band

comment:2 by mlennert, 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 neteler, 7 years ago

Milestone: 7.0.67.0.7

by mlennert, 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 mlennert, 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:5 by martinl, 6 years ago

Still relevant?

comment:6 by mlennert, 6 years ago

Milestone: 7.0.77.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:7 by neteler, 5 years ago

Milestone: 7.8.07.8.1

Ticket retargeted after milestone closed

comment:8 by neteler, 5 years ago

Milestone: 7.8.17.8.2

Ticket retargeted after milestone closed

comment:9 by neteler, 5 years ago

Milestone: 7.8.2

Ticket retargeted after milestone closed

comment:10 by neteler, 5 years ago

Milestone: 7.8.3
Note: See TracTickets for help on using tickets.