Re: [Qgis-developer] Re: something went terribly wrong with reprojected rasters in the last 72 hours
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
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
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
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