Giter Site home page Giter Site logo

Comments (8)

hercek avatar hercek commented on June 16, 2024 1

It is already there but named badly in UI. Use Lap speed/pace instead of Average speed.

Lap speed/pace is actually "average speed of the last interval" despite the fact that it is named "Lap speed/pace". If I understand the code correctly then there is no lap detection in it. The only way how laps can be taken into account is if you set announcer interval to distance which is the same as the length of your lap. There should not be any mention of laps in the UI since there is no lap detection. On the other side, I would not appreciate if "interval" adjective is added to the UI since it gets too verbose.

My opinion is that the settings tab should support only two types of measurements:

  1. Average/total quantities which would refer to the whole track.
  2. Quantities without any qualifier (adjective added to it) in which case they would refer to the last interval: e.g. if it is "Speed" then it would mean average speed of the last interval.

from opentracks.

timror avatar timror commented on June 16, 2024

Tnx for the explanation. I've tried the app with the following settings (sorry it's in Dutch)

Imagepipe_2

Now I only hear the elapsed time, but no spoken feedback of the speed of the interval. Is it possible the voice output is broken somewhere with that specific setting?

Returning to the previous setting (avg /speed) does give me voice output, so that setting works as expected.

from opentracks.

hercek avatar hercek commented on June 16, 2024

Interesting. It works for me when I'm using distance interval instead fo time interval and only "lap" speed is reported. I'll try something more similar to your settings.

Do you have "Use device speaker" checked?
Do you use "distance and speed" Bluetooth sensor?
Does GPS properly record your track? I.e. whey you ask to show a track on a map, does it show correctly?
What version do you use?

from opentracks.

timror avatar timror commented on June 16, 2024

Do you have "Use device speaker" checked?

It is unchecked. Normally I hear the feedback through my earpieces (aux port)

Do you use "distance and speed" Bluetooth sensor?

I don't use an external Bluetooth device. I only use my phone GPS.

Does GPS properly record your track? I.e. whey you ask to show a track on a map, does it show correctly?

Yes, works very well with KML export to OrganicMaps

What version do you use?

v.4.9.9

from opentracks.

hercek avatar hercek commented on June 16, 2024

OK, I tested how this behaves today on version 4.10.0. It is very likely the behaviour is the same as in 4.9.9.

You can change this issue to a bug report:
"announcer's interval based quantities are not reported when there is only time interval setup"

The likely actual reason is that these quantities are computed only based on the distance interval set in the announcer. If one has setup only time based interval then interval ("lap") based quantities are not computed at all and therefore not reported at all. If you would have setup both time and distance based interval then the interval ("lap") based quantities would be stalled during time based report; i.e. the would be reported as they were computed in the last distance interval.

If you are not allowed to do this change then create a new issue and link to this one.

Somebody may resolve this eventually. You can use a distance based interval in the meantime. Set "Time interval" to "Off" and set some distance in "Distance interval" in the Announcer settings.

from opentracks.

hercek avatar hercek commented on June 16, 2024

One way how to resolve this issue without changing java code is to keep "Lap" string in the "distance interval" based announcer quantities and only change "Distance interval" string in the settings to "Lap length". I quite like this option. It has my vote.

Though it would be handy to be able to set any lap length (and not only the preselected ones) in this case.

from opentracks.

dennisguse avatar dennisguse commented on June 16, 2024

@timror @hercek pretty good discussion!
Kudos!

@hercek can you create a PR for changing the strings?
The idea sounds pretty good :D

from opentracks.

hercek avatar hercek commented on June 16, 2024

I can change "Distance interval" string in options to "Lap length" string. It will make things more clear in the options.

But later. I have currently unfinished version of PR for #1805 and do not want to mix many things together.

from opentracks.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.