[Marble-devel] Re: Review Request: introduce optimized code path for Mercator projection

Torsten Rahn rahn at kde.org
Thu Dec 23 02:22:09 CET 2010



> On 2010-12-13 20:27:26, Torsten Rahn wrote:
> > /trunk/KDE/kdeedu/marble/src/lib/FastMercatorTextureMapper.h, line 20
> > <http://svn.reviewboard.kde.org/r/6109/diff/1/?file=42508#file42508line20>
> >
> >     "Fast" doesn't really provide an idea how this works.
> 
> Bernhard Beschow wrote:
>     Right. IMO ScalingTextureMapper would be a better name. Last but not least it can also be applied in the same way to the Equirectangular projection.
> 
> Bernhard Beschow wrote:
>     I renamed it to TileScalingTextureMapper. Ok?

TileScalingTextureMapper is ok :-)


- Torsten


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://svn.reviewboard.kde.org/r/6109/#review9241
-----------------------------------------------------------


On 2010-12-22 22:33:36, Bernhard Beschow wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://svn.reviewboard.kde.org/r/6109/
> -----------------------------------------------------------
> 
> (Updated 2010-12-22 22:33:36)
> 
> 
> Review request for marble.
> 
> 
> Summary
> -------
> 
> This patch introduces an optimized code path for the case that both the tile projection and the map projection are of type Mercator. In this very common case, the tiles can be scaled rather than reprojected, promising a significant speedup. Moreover, it seems to avoid crashes in high zoom levels on the N900.
> 
> 
> Diffs
> -----
> 
>   /trunk/KDE/kdeedu/marble/src/lib/AbstractScanlineTextureMapper.h 1208706 
>   /trunk/KDE/kdeedu/marble/src/lib/AbstractScanlineTextureMapper.cpp 1208706 
>   /trunk/KDE/kdeedu/marble/src/lib/CMakeLists.txt 1208706 
>   /trunk/KDE/kdeedu/marble/src/lib/TextureLayer.cpp 1208706 
>   /trunk/KDE/kdeedu/marble/src/lib/TileScalingTextureMapper.h PRE-CREATION 
>   /trunk/KDE/kdeedu/marble/src/lib/TileScalingTextureMapper.cpp PRE-CREATION 
> 
> Diff: http://svn.reviewboard.kde.org/r/6109/diff
> 
> 
> Testing
> -------
> 
> I have measured speedups of up to 2.7 on my system. This seems to depend on the QImage::Format, so format conversion should be avoided in a future version if possible.
> The image quality is roughly of outline quality when a discrete zoom level isn't hit.
> 
> I get crashes on my Netbook, but not on my other machine. The error message is: "QPaintDevice: Cannot destroy paint device that is being painted". I've got no idea what's going on here. Please help.
> 
> 
> Thanks,
> 
> Bernhard
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://mail.kde.org/pipermail/marble-devel/attachments/20101223/1180fd70/attachment-0001.htm 


More information about the Marble-devel mailing list