irusanov / ZenTimings

https://zentimings.com/
GNU General Public License v3.0
345 stars 18 forks source link

"," and "." in voltages #51

Open SnIPeRSnIPeR opened 2 months ago

SnIPeRSnIPeR commented 2 months ago

In 1.32 I see "," and "." in voltages, instead of just "."...

2024-08-09_17-39-50

Can you fix that?

irusanov commented 2 months ago

Something with the locale, will have to take a look and make sure all values use the same formatting. Does it happen with the recent version only? I have changed the place some values are read from, maybe that is contributing to the issue. Everything looks fine on my machine, but I have en-US with custom date and clock format. I think it comes from the toString() method, but not sure why it differs for different values.

SnIPeRSnIPeR commented 2 months ago

Something with the locale, will have to take a look and make sure all values use the same formatting. Does it happen with the recent version only? I have changed the place some values are read from, maybe that is contributing to the issue. Everything looks fine on my machine, but I have en-US with custom date and clock format. I think it comes from the toString() method, but not sure why it differs for different values.

Yep, I see this only with the recent version :)

1.31, for example: ZenTimings_Screenshot_28626332 1609445_Работает без ошибок

irusanov commented 2 months ago

Ok, it seems the values that now are provided by the dll and not the main app are with different CultureInfo (sort of an locale formatting).