[Marble-devel] Review Request 109379: ElevationProfile: support for gpx-tracks

[Bernhard Beschow]

> On April 15, 2013, 5:17 p.m., Bernhard Beschow wrote:
> > Thanks for the patch, looks really nice! Two nitpicks, though: I get every track listed twice in the menu, I wonder why this happens. Morevoer, the elevation profile doesn't display the relevant interval of V-shaped tracks when the bottom edge of the V isn't visible. When these issues are fixed, I'd give you a "Ship it". :)
> 
> Roman Karlstetter wrote:
>     Could you please send me a file for which this happens? Because I can't reproduce this with the files I have. 
>     The same holds for the second point (even if I'm not completely sure that I understood it correctly).
> 
> Bernhard Beschow wrote:
>     I'll check this later. Could you please add "marble" to the group of reviewers, please? It seems as if our conversation is private, i.e. isn't distributed to Marble's developer mailing list. Thx!

How to reproduce that a .gpx file is shown more than once:
1) Start Marble (KDE version, latest master)
2) Enable the elevation profile float item
3) Open examples/gpx/mjolby.gpx from the Marble source code
Observe that mjolby.gpx is present more than once in the float item's context menu (6 times in my case)

Continue with reproducing an empty float item although the track is clearly visible:
4) Enable "Zoom to viewport" in the context menu
5) Center on Vikingstad on tile level 14
6) Keeping that zoom level, follow the track along the highway in western direction until road "206" is crossed
7) Follow the track along this road to Skänninge
Observe that the float item eventually says: "Not enough points in the viewport" although the track is clearly visible.


- Bernhard


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109379/#review31110
-----------------------------------------------------------


On April 16, 2013, 5:19 p.m., Roman Karlstetter wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109379/
> -----------------------------------------------------------
> 
> (Updated April 16, 2013, 5:19 p.m.)
> 
> 
> Review request for Marble and Bernhard Beschow.
> 
> 
> Description
> -------
> 
> Changes to the elevation profile plugin, it's now possible to display the height profile for a GPX-track opened from file.
> 
> I made a new classes for
> a) providing the data, one for routes and one for tracks. They are in the same file, should this be split into multiple files?
> b) the contextmenu, because it has some members which would only clutter ElevationProfileFloatItem. It is a friend class, is this ok?
> 
> Comments please :) 
> 
> 
> Diffs
> -----
> 
>   src/plugins/render/elevationprofilefloatitem/ElevationProfileContextMenu.h PRE-CREATION 
>   src/plugins/render/elevationprofilefloatitem/CMakeLists.txt 872e5e1 
>   src/plugins/render/elevationprofilefloatitem/ElevationProfileContextMenu.cpp PRE-CREATION 
>   src/plugins/render/elevationprofilefloatitem/ElevationProfileDataSource.h PRE-CREATION 
>   src/plugins/render/elevationprofilefloatitem/ElevationProfileDataSource.cpp PRE-CREATION 
>   src/plugins/render/elevationprofilefloatitem/ElevationProfileFloatItem.h cb3bff1 
>   src/plugins/render/elevationprofilefloatitem/ElevationProfileFloatItem.cpp 7cb3428 
> 
> Diff: http://git.reviewboard.kde.org/r/109379/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Roman Karlstetter
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/marble-devel/attachments/20130425/c5e98e7f/attachment.html>