Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Short distances and elevation with statute units #626

Open
gerhardol opened this issue Aug 21, 2017 · 8 comments
Open

Short distances and elevation with statute units #626

gerhardol opened this issue Aug 21, 2017 · 8 comments
Labels

Comments

@gerhardol
Copy link
Collaborator

RU is currently presenting km/m also with miles settings in some situations.
I have suggested changing that in PR #624
However, Short distances (less than a mile) are presented as 'm' still.
Changing that will require translation changes, so there are some steps required.

At this point, I would prefer to not make the setting configurable.
Also, I would not like to present in combined units, like miles and yards.

How to present short distances, in yard or feet or even keep meters?
There is no presentation of elevation right now, if that is added (for instance in graphs), is it OK to use the short distance setting?

@davidedelvento
Copy link

I think there should be 3 options: metric, imperial (aka statute) or both, because different people have different needs. Not sure why you want to make this not-configurable: it must be.

Also, it is not clear why short distances should be treated differently than longer distances. In my opinion everything should be treated the same. Perhaps elevation might be treated differently (I prefer not, but I can see the point), but why short distances?

@gerhardol
Copy link
Collaborator Author

I want to keep the current option: Km (should be named metric) and Miles (should be named Imperial or Statue).
What is your third option?

Short distances are used to configure workouts and get better audio cues (re there cues for how much remains?)
Not something I use myself.

I changed the special case for '0' to display as km/mi to make it more natural when creting manual activities.

@davidedelvento
Copy link

The proper spelling is statute not statue, see e.g. https://en.wikipedia.org/wiki/Mile#Statute_mile

My third option is both, i.e. being able to get some/all the distance/speed in both units.

Still, I am not sure why short distances should be configured different than long distances, let alone hard coded to (say) metric and confuse people who only know imperial (or viceversa).

@gerhardol gerhardol changed the title Short distances and elevation with statue units Short distances and elevation with statute units Aug 28, 2017
@gerhardol
Copy link
Collaborator Author

A temporary (?) solution could be to only present Imperial (avoiding to misspell statute!), in miles decimal only, then it is like @daviddelvento suggests?

@davidedelvento
Copy link

I'm actually suggesting this being (manually) localized like everything else. Let's aside my "third option" for the the time being because I think it's too advanced and for a smaller user base (even if that user base includes myself, so I'm very interested into it).

If we do what you're suggesting I think it'd be confusing for metric people who select metric system everywhere in their phones to see these numbers in miles.

Now if localize short distances and elevation is too complicated to do (why????), feel free to do what you said.

@gerhardol
Copy link
Collaborator Author

If we do what you're suggesting I think it'd be confusing for metric people who select metric system everywhere in their phones to see these numbers in miles.
Of course only short distances in miles if distances are set in miles, metric get m still.

But if this is to be customized, how to present it?
Will short distance be understood?
If not, then I prefer to have this hardcoded.

Unit (new subpage):
Distance:
km, miles
Short distance:
km, miles, m (yard, feet?)

@davidedelvento
Copy link

I'm suggesting:

Units (page)
Distance (one element): metric, imperial, (and ideally the option to pick both)

and to be applied for both long or short distance with the proper adjustments made automatically.

For example, if one sets metric:
Total distances (both long and short) and altitude can be in m, whereas pace will be in s/km (not in s/m)

If one sets imperial:
Totals can be miles for long distances, whereas for short distances (=less than one mile) and altitudes it will be feet.

Optionally, if not too much of a hassle, when one sets both imperial and metric: both the metric and imperial numbers will be shown in live numbers, tables etc. Unless you plan to considerably increase the font, this will not be a problem since most of the screen of my phone (the smallest screen with the lowest resolution I could buy) is mostly empty.

@gerhardol
Copy link
Collaborator Author

Then feet it is
Should add cues for yards too
(Not sure if I will change names to imperial/Metric, see what translation is needed. Someone else may change too).

"Double" printouts are difficult to handle, it would require a very special setup.
What should be done is to make it possible to configure the fields that are presented.
RU is now a data logger with more advanced audio cues, not something too look at while running.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants