Opened 7 years ago

Closed 5 years ago

#2041 closed defect (fixed)

wxgui: error attempting to display imported raster map that was imported as three bands

Reported by: mlennert Owned by: grass-dev@…
Priority: normal Milestone: 7.0.0
Component: wxGUI Version: svn-trunk
Keywords: raster import display Cc:
CPU: Unspecified Platform: Unspecified

Description

When you use the File->Import raster data->Common formats import wizard frontend to r.in.gdal to import a raster file that will be split into .red, .green, .blue bands on import, and you have "Add imported layers to layer tree" checked, then you get an error message since the GUI tries to zoom to and display a map with a name without the band extensions.

To reproduce, use the wizard to import any jpg, png, tiff or other raster format that gets split into the three color bands.

Change History (6)

comment:1 Changed 6 years ago by wenzeslaus

As a temporary solution, GUI currently contains a text that says that Add layers... may not work with multiple bands.

comment:2 in reply to:  1 ; Changed 6 years ago by mlennert

Replying to wenzeslaus:

As a temporary solution, GUI currently contains a text that says that Add layers... may not work with multiple bands.

I guess to get a "real" solution, the dialog would have to run gdalinfo first to check whether there are multiple bands in the file ? Either using the python gdal api (RasterCount?) or using some basic grep:

gdalinfo MyRaster.File | grep Band | wc -l

comment:3 in reply to:  2 Changed 6 years ago by mmetz

Replying to mlennert:

Replying to wenzeslaus:

As a temporary solution, GUI currently contains a text that says that Add layers... may not work with multiple bands.

I guess to get a "real" solution, the dialog would have to run gdalinfo first to check whether there are multiple bands in the file ?

This is not enough, you would also need to check if the band option or the k flag is used with r.in.gdal.

In order to keep things sufficiently simple (which the wxGUI not longer is, but anyway), I would rather suggest to test if the output raster exists, if not, the GUI should not attempt to display it.

comment:4 Changed 6 years ago by neteler

See also bug #1716

comment:5 Changed 5 years ago by wenzeslaus

Please test the current trunk. See Martin's r64372 and r64371 and Anna's r64570 (r64598).

comment:6 in reply to:  5 Changed 5 years ago by mlennert

Resolution: fixed
Status: newclosed

Replying to wenzeslaus:

Please test the current trunk. See Martin's r64372 and r64371 and Anna's r64570 (r64598).

yes, this is fixed. Thank you for the reminder !

Note: See TracTickets for help on using tickets.