-
Notifications
You must be signed in to change notification settings - Fork 31
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
Some cultures use ',', not '.', for the decimal point. #21
Comments
I am in one of those cultures that use a comma as decimal separator but haven't found it to be an issue. |
Almost all of the numerical options.
eg. `>trade.py run --ly=16.25 --empty=19.41 --jum=2 --lsp=12.5`
…On Sun, Feb 3, 2019 at 11:10 AM Peter Magnusson ***@***.***> wrote:
I am in one of those cultures that use a comma as decimal separator but
haven't found it to be an issue.
If it's for display only I would say that is doesn't matter. It's would be
nice to have but not a must have.
Entry might be another thing since comma is a key on the numerical part of
the keyboard and dot isn't.
Where/When would you need to enter decimals?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#21 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ABpz4BjmXaljsTegCKufvtPz9eOBQP-kks5vJyYNgaJpZM4WWXnN>
.
--
mu'o mi'e .aionys.
.i.e'ucai ko cmima lo pilno be denpa bu .i doi.luk. mi patfu do zo'o
(Come to the Dot Side! Luke, I am your father. :D )
|
Good point. I'v been lazy and been rounding usually down. |
Well, like you said, it's not a big issue. I may or may not get to it at some point in the future, or someone else who really cares and knows what they're are doing might see this issue and decide to make a PR, but it's low priority for a reason. |
Maybe look into making TD able to handle that.
The text was updated successfully, but these errors were encountered: