Opened 3 years ago

Last modified 5 weeks 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.2
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 3 years ago.
patch to allow import of any band
i_in_spotvgt_any_band.diff (8.1 KB) - added by mlennert 22 months ago.
patch that allows importing any band (one by one) - provided by Tudor-Emil Coman

Download all attachments as: .zip

Change History (10)

comment:1 Changed 3 years ago by mlennert

Milestone: 7.0.6

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

Changed 3 years ago by mlennert

Attachment: i_in_spotvgt_all_bands.diff added

patch to allow import of any band

comment:2 Changed 3 years ago 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.

comment:3 Changed 22 months ago by neteler

Milestone: 7.0.67.0.7

Changed 22 months ago by mlennert

Attachment: i_in_spotvgt_any_band.diff added

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

comment:4 Changed 22 months ago 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 ?

comment:5 Changed 8 months ago by martinl

Still relevant?

comment:6 Changed 7 months ago by mlennert

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 Changed 3 months ago by neteler

Milestone: 7.8.07.8.1

Ticket retargeted after milestone closed

comment:8 Changed 5 weeks ago by neteler

Milestone: 7.8.17.8.2

Ticket retargeted after milestone closed

Note: See TracTickets for help on using tickets.