Opened 4 years ago

Last modified 7 months ago

#3037 new enhancement

r.import - output name should not be required with -e flag

Reported by: guano Owned by: grass-dev@…
Priority: normal Milestone: 7.6.2
Component: wxGUI Version: 7.0.3
Keywords: import Cc:
CPU: Unspecified Platform: All

Description

In r.import (import with reprojection), the -e flag should ignore if there is an output name defined, much like the 'remove mask' option.

Also, the output name could be set to be the same as the input by default, like in r.in.gdal.

Change History (12)

comment:1 Changed 4 years ago by annakrat

Try r68466.

comment:2 Changed 3 years ago by martinl

Backport candidate?

comment:3 in reply to:  2 Changed 3 years ago by annakrat

Replying to martinl:

Backport candidate?

Yes, but someone please confirm it works first.

comment:4 Changed 3 years ago by martinl

Tested, seems to work.

comment:5 Changed 3 years ago by martinl

Milestone: 7.0.57.3.0

comment:6 Changed 3 years ago by martinl

There is probably still a bug, when I run

r.import in=/opt/geodata/piemont/landsat_4_5_7_8/LE71940292015195NSG00_sr_band5_piemont.tif -e

in the location which match the input data, the resolution is not printed and moreover output map (rimport_tmp) is not deleted.

r.import in=/opt/geodata/piemont/landsat_4_5_7_8/LE71940292015195NSG00_sr_band5_piemont.tif -e 
Proceeding with import of 1 raster bands...
Importing raster map <rimport_tmp>...
0..3..100
g.list raster pat=rimport_tmp
rimport_tmp

comment:7 Changed 3 years ago by martinl

Milestone: 7.3.07.0.5

comment:8 Changed 3 years ago by annakrat

The question is what should the r.import do when no reprojection is needed and user uses -e flag. Should the raster be imported, then we print its resolution and delete it? That's probably consistent, but seems weird.

comment:9 Changed 3 years ago by neteler

Milestone: 7.0.57.0.6

comment:10 Changed 21 months ago by neteler

Milestone: 7.0.67.0.7

comment:11 Changed 7 months ago by martinl

Still relevant?

comment:12 Changed 7 months ago by martinl

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