UTF-8 is now explicitly used in the code. - #254
Conversation
This should allow Samsung devices to display the 12V chart again.
|
OK… first of all, please explain the "robust timestamp parsing" change. Why would that be needed, how could the record structure be changed in a way that makes this necessary? If so, why would no other records be affected? Second… I cannot find any source for the assumed UTF-8 issue on Samsung devices. You wrote:
WTF? If Samsung would change the library & API defaults in a way you need to now explicitly specify UTF-8 (which is the defined default for all Android APIs), most Apps would break immediately. I cannot believe that has any true core. Please supply a link to the source of that info. Is this change actually based on any real data and testing? |
|
If this gets merged, I look forward to seeing if it addresses the missing 12v Battery History issue [which may be limited to Samsung android devices]. |
|
Zorg, any actual source for that Samsung allegation? I've tried again, and all I can find are some baseless AI hallucinations. |
|
The technical adjustments implemented for the "Samsung Fix" address three common challenges in Android development that frequently occur on Samsung devices due to One UI firmware customizations and specific hardware (Exynos) drivers: 1. Character Set Conflicts (UTF-8)
2. Graphical Rendering Errors (Software Layer)
3. Regional Date Formats (Locale.US)
These points are standard "best practices" that ensure the app remains robust across various manufacturer implementations (Samsung vs. Pixel vs. others). |
|
On point 1:
That simply cannot be true, as explained above. Let's check the docs again: https://developer.android.com/reference/java/nio/charset/Charset#defaultCharset()
It's always UTF-8, with no exceptions. What is the actual source for the allegation of Samsung diverging from the Android platform specs? Please don't answer with another LLM generated text, give a link to an actual site discussing this. And btw, if that would be an issue for Samsung devices, it wouldn't be restricted to the 12V history data, and it wouldn't happen to occur only after April 2026 -- Samsung devices are all over the place, and no issues of this kind have been reported before. On point 2: Again, why wouldn't that affect all charts on the devices having the issue?
This is clearly an AI hallucination, there is no mentioning of any Samsung / Exynos / GPU issues in https://github.com/PhilJay/MPAndroidChart, neither in issues nor in pull requests or commits. Same for the Android HW accel docs. The only place I found discussing this is the PlayCanvas forum, they found an issue initially thought to be Samsung specific, that later turned out to be specific for the Mali G52 chipset. So we should first check if the devices affected have that chipset. @MylesTwete please check your hardware specs for the GPU chipset used. On point 3: That's correct and a valid change. But it still doesn't apply to the actual issue, as according to Myles there was no region/language change on his devices. |
This should allow Samsung devices to display the 12V chart again.
#253