[Qgis-developer] Re: something went terribly wrong with reprojected rasters in the last 72 hours

[Radim Blazek]

The patch is simple, we have to wait for new repository

--- src/core/qgsrasterprojector.cpp     (revision 15861)
+++ src/core/qgsrasterprojector.cpp     (working copy)
@@ -130,7 +130,7 @@
   mSrcExtent = QgsRectangle( myPoint.x(), myPoint.y(), myPoint.x(),
myPoint.y() );
   for ( int i = 0; i < mCPRows; i++ )
   {
-    for ( int j = 1; j < mCPCols - 1; j++ )
+    for ( int j = 0; j < mCPCols; j++ )
     {
       myPoint = mCPMatrix[i][j];
       mSrcExtent.combineExtentWith( myPoint.x(), myPoint.y() );

Radim

On Mon, May 2, 2011 at 1:29 PM, Mathieu Pellerin <nirvn.asia at gmail.com> wrote:
> Greetings Radim Blazek,
>
> I'm forwarding you an email sent to the qgis-developer list hoping it can
> speed up a major raster rendering issue emerging days before the 1.7
> release.
>
> Please find attached to this email a rendered map showing the issue. I
> believe it might have been caused by r15845 or r15846, both applied by you a
> few days ago. Others seem to draw the same conclusions
> (http://lists.osgeo.org/pipermail/qgis-developer/2011-May/014148.html).
>
> Hope it can be of help to you in resolving this issue.
>
> Thanks for all the hard work you and the other developers are putting into
> QGIS, it's an amazing piece of software.
>
> Math
>
> ---------- Forwarded message ----------
> From: Mathieu Pellerin <nirvn.asia at gmail.com>
> Date: Mon, May 2, 2011 at 11:27 AM
> Subject: something went terribly wrong with reprojected rasters in the last
> 72 hours
> To: qgis-developer at lists.osgeo.org
>
>
> Greetings,
>
> QGIS' trac login is timing out on me so going through the mailing list until
> it allows me to file in a proper issue report.
>
> A change applied on QGIS within the last 72 hours resulted in messy
> rendering of reprojected rasters. From what I can see, starting at pixel's
> column 50%+1, QGIS just keeps repainting the same pixels again and again
> (see attached rendering).
>
> This should probably be fixed prior to releasing of 1.7 : )
>
> Cheers.
>
> Math
>
>