Comment 10 for bug 1035343

Revision history for this message
Joshua Judson Rosen (rozzin) wrote : Re: [Bug 1035343] Re: Feature: Add route planning support to foxtrotgps (Patch included)

"Dr. Tilmann Bubeck" <email address hidden> writes:
>
> I made all requested changes and uploaded the attached patch which is
> based on lp:~rozzin/foxtrotgps/route-planning.cleanup

Great--thanks!

> * waypoints/trackpoints/routepoints? Well, I think we are planing a
> route consisting of routepoints (rtept). However, some web pages speak
> of routes consisting of waypoints, so I think this is not very clear
> by definition. Regardless of what we write into our generated GPX
> files we are able to read "rtept" and also "waypt", but not "trackpt"
> which is for tracking a route which you already travelled. So I did
> not change anything, because I do think that it is consistent.

Agreed.

> In addition I reordered the entries of the "route & wp" menu. I moved
> all entries dealing with single points at the top of the menu and all
> entries dealing with all waypoints (=routes) to the end.

Regardless of the ordering of the menu-items, I think it ends up being
confusing because each of "route" and "WP" both now mean multiple,
different things:

    * The pre-existing "get route" just draws a non-editable route
      (like you'd get when loading a *track* from a GPX file).
    * Your new "load route from GPX" loads a series of editable
      routepoints, and has nothing to do with "get route"
      (likewise "save route as GPX" has nothing to do with the `route'
       generated by "get route"; and "clear route" clears only
       your `route of waypoints', not routes from "get route").

    * The pre-existing "set WP" and "unset WP" do something
      orthogonal to your "add WP", "delete WP", and "insert WP before"
      operations.

One thought I had on this was we could try to disambiguate the different
features by moving them to different menus (e.g.: "route planning"
vs. "route finding").... But, based upon your explanation above,
I wonder if the best option is actually just to label your new operations
as being `routepoint' operations rather than `route' or `waypoint'
operations. Something like:

    * add routepoint
    * insert routepoint before this
    * delete routepoint
    * clear routepoints
    * save routepoints to GPX route
    * save routepoints to TomTom ITN

Or is that also confusing? Would it be better to do something like:

    * clear planned route
    * save planned route to GPX
    * save planned route to TomTom ITN

We could also resolve the "WP" conflict by renaming the historic WP
menu-items, e.g.:

    * set target
    * unset target

Thoughts?

I guess I should pose the question to the foss-gps list, as well....

--
"Don't be afraid to ask (λf.((λx.xx) (λr.f(rr))))."