yan/archived-matedroid
1
0
Fork 0
mirror of https://github.com/vide/matedroid.git synced 2026-01-20 00:03:17 +08:00
Code Issues Packages Projects Releases Wiki Activity

fix(dashboard): don't double-convert tire pressure when unit is PSI (#49)

Browse Source 
TeslamateAPI returns pressure already in the user's preferred unit.
Remove incorrect bar→PSI conversion that caused 42 PSI to display as 610 PSI.

Fixes #46
This commit is contained in:
Davide Ferrari
2026-01-13 07:32:12 +01:00
committed by GitHub
parent 70ae969d0c
commit 96d67e9cb9
1 changed files with 6 additions and 9 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
app/src/main/java/com/matedroid/domain/model/UnitFormatter.kt
View File

@@ -11,7 +11,6 @@ object UnitFormatter {
// Conversion constants // Conversion constants
private const val KM_TO_MI = 0.621371 private const val KM_TO_MI = 0.621371
private const val MI_TO_KM = 1.60934 private const val MI_TO_KM = 1.60934
private const val BAR_TO_PSI = 14.5038
private const val WH_PER_KM_TO_WH_PER_MI = 1.60934 private const val WH_PER_KM_TO_WH_PER_MI = 1.60934
/** /**
@@ -75,15 +74,13 @@ object UnitFormatter {
} }
/** /**
* Format pressure value with appropriate unit label * Format pressure value with appropriate unit label.
* Note: TeslamateAPI returns pressure already in the user's preferred unit,
* so no conversion is needed - just format and add the label.
*/ */
fun formatPressure(bar: Double, units: Units?, decimals: Int = 1): String { fun formatPressure(value: Double, units: Units?, decimals: Int = 1): String {
return if (units?.unitOfPressure == "psi") { val unit = if (units?.unitOfPressure == "psi") "psi" else "bar"
val psi = bar * BAR_TO_PSI return "%.${decimals}f %s".format(value, unit)
"%.${decimals}f psi".format(psi)
} else {
"%.${decimals}f bar".format(bar)
}
} }
/** /**