Opened 12 years ago

Closed 9 years ago

#172 closed defect/bug (fixed)

replace "soon" by a real distance estimate

Reported by: kiddo Owned by: KaZeR
Priority: major Milestone: version 1.0
Component: core Version: git master
Severity: Keywords: usability
Cc: nick@…

Description (last modified by akashihi)

soon is useless. The driver needs a real distance/time estimate announcement, which navit should have no problem providing since it already does for other announcements. 2 3 This also entails a sub-bug: if the user is told "soon" and that navit means "in 5 street corners", the user will have no idea if the street corner/traffic light he sees in 10 meters is the one to be considered "soon".

Attachments (1)

issue172.patch (357 bytes) - added by kiddo 12 years ago.
one liner patch that fixes the problem for good

Download all attachments as: .zip

Change History (9)

Changed 12 years ago by kiddo

one liner patch that fixes the problem for good

comment:1 Changed 12 years ago by KaZeR

  • Milestone set to version 0.1.0
  • Owner changed from somebody to KaZeR
  • Priority changed from critical to major
  • Status changed from new to assigned

I personnaly agree about the fact that 'soon' is useless. But i'll have to check with cp15 if he's ok about this change.

comment:2 Changed 12 years ago by KaZeR

It is intended to make it configureable.

comment:3 Changed 11 years ago by Kiddo

well, in the meantime, having the (I think) "saner" default that this patch provides is better. The option to control it (though I have no idea why someone would want navit to give uncertain statements like "soon") could come later.

comment:4 Changed 10 years ago by akashihi

  • Description modified (diff)
  • Milestone changed from version 0.2.0 to To be discussed - Give your opinion!

comment:5 Changed 10 years ago by akashihi

  • Description modified (diff)

In my opinion, 'soon' is usefull. It warns you that in the near future you'll get detailed description of the next maneuvre.

Oops, missed entry fields

comment:6 Changed 10 years ago by number6

I agree with OP - Replace "Soon" with a real distance estimate.

comment:7 Changed 9 years ago by number6

  • Cc nick@… added

To back track on myself, "Soon" can be configured in navit.xml

<announcement level="2" distance_metric="200"/> Is an example of the line that can be edited.

comment:8 Changed 9 years ago by korrosa

  • Resolution set to fixed
  • Status changed from assigned to closed

To clarify Number6's point:

  • <announcement level="2" distance_metric="200"/> specifies the distance (metres) before the manoeuvre when "soon" is used.
  • <announcement level="1" distance_metric="100"/> specifies the distance (metres) before the manoeuvre when "in X metres" is used.
  • <announcement level="0" distance_metric="50"/> specifies the distance (metres) before the manoeuvre when "now" is used.

As such, the functionality is available to enable/disable the "now" announcement in Navit. If you wanted Navit to give two announcements with "in X metres", then try applying the patch.

Note: See TracTickets for help on using tickets.