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

2011-05-07 Thread Giovanni Manghi
On Sat, 2011-05-07 at 12:31 +0700, Mathieu Pellerin wrote:
 Radim,
 
 Any chance you can apply this quick fix to a 1.7 build prior to final
 release?

by the way the patch seems to work fine.

cheers

-- Giovanni --

___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/qgis-developer


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

2011-05-07 Thread Tim Sutton
I've committed Radims patch both to master and to the release branch:
commit:a87cb60

Regards

Tim


On Sat, May 7, 2011 at 3:46 PM, Giovanni Manghi
giovanni.man...@gmail.com wrote:
 On Sat, 2011-05-07 at 12:31 +0700, Mathieu Pellerin wrote:
 Radim,

 Any chance you can apply this quick fix to a 1.7 build prior to final
 release?

 by the way the patch seems to work fine.

 cheers

 -- Giovanni --

 ___
 Qgis-developer mailing list
 Qgis-developer@lists.osgeo.org
 http://lists.osgeo.org/mailman/listinfo/qgis-developer




-- 
Tim Sutton - QGIS Project Steering Committee Member (Release  Manager)
==
Please do not email me off-list with technical
support questions. Using the lists will gain
more exposure for your issues and the knowledge
surrounding your issue will be shared with all.

Visit http://linfiniti.com to find out about:
 * QGIS programming and support services
 * Mapserver and PostGIS based hosting plans
 * FOSS Consulting Services
Skype: timlinux
Irc: timlinux on #qgis at freenode.net
==
___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/qgis-developer


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

2011-05-06 Thread Mathieu Pellerin
Radim,

Any chance you can apply this quick fix to a 1.7 build prior to final
release?

Math
On May 2, 2011 6:56 PM, Radim Blazek radim.bla...@gmail.com wrote:
 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.a...@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.a...@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@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


___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/qgis-developer


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

2011-05-02 Thread 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.a...@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.a...@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@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


___
Qgis-developer mailing list
Qgis-developer@lists.osgeo.org
http://lists.osgeo.org/mailman/listinfo/qgis-developer