incorrect Min.Y and Max.Y

[hmzhu]

Min.Y and Max.Y in rdc file are reversed when GetGeoTransform() fails, which leads to failure of displaying the resulting image in Idrisi software. See screenshot Stockholm_old.jpg snapped from Idrisi. Please pay attention to the area in the red box.

Solution: Correct the code in lines 987 - 990 in Idrisi driver in GDAL1.10.1. In addition, since Idrisi has decided the starting value will be 0, rather than 0.5, each 0.5 should be removed too, which also affects MinX and MaxX.

Fixing actually is done in the revised trunk of Idrisi driver I submitted last year, in which the default values defined in lines 790 – 793(the numbers are in your version) will be used if GetGeoTransform() fails. Result please see screenshot stockholm.jpg.

[hmzhu]

This is the result from GDAL1.10.1

[hmzhu]

This is the result from GDAL with revised Idrisi driver

[Even Rouault]

I'm not sure to understand what you want : backport changes from trunk into 1.10 branch ? Or is there still an issue with trunk ?

Does it relate with http://trac.osgeo.org/gdal/changeset/26033/trunk/gdal/frmts/idrisi / #4980 ?

[hmzhu]

Replying to rouault:

I'm not sure to understand what you want : backport changes from trunk into 1.10 branch ? Or is there still an issue with trunk ?

Does it relate with http://trac.osgeo.org/gdal/changeset/26033/trunk/gdal/frmts/idrisi / #4980 ?

These bugs have been fixed in the trunk(#4980). But Idrisi driver being used in GDAL1.10.1 is a old version(source from this link:http://trac.osgeo.org/gdal/wiki/DownloadSource). It doesn't reflect changes made in the trunk. So old bugs come up when using GDAL1.10.1.

Hongmei

[Even Rouault]

ok, so this is a duplicate of #4980. Closing it as such.

Frank who committed the changes in trunk didn't feel confident backporting them in 1.10 branch where we usually do only tiny or critical fixes for regressions after it has been forked off from trunk, but not improvements or "nice to have" changes. Your changes will be available in the next major GDAL version.