fdim-controller icon indicating copy to clipboard operation
fdim-controller copied to clipboard

Tire display order

Open bmwhitetx opened this issue 4 years ago • 0 comments

In the case statement for setting up the display, The HU_STOCK and two HU_CHINESE's both have the rear tire pressures listed as RL first and RR second which makes sense and matches the front tire order defined in preceding line.

However the HU_AFTERMARKET has the order reversed: RR is listed on the left and RL is on on the right. (lines removed for readablilty)

      case HU_AFTERMARKET:
        displayText(1, tirePressure[TIRE_FL].padRight(pressurePadding) + rpmMessage + rpm.padRight(4) + F(" E:") + temperature.padRight(3) + " " + tirePressure[TIRE_FR].padRight(pressurePadding));
        displayText(2, tirePressure[TIRE_RR].padRight(pressurePadding) + tireTempMessage + tireTemperature.padRight(3) + " " + tirePressure[TIRE_RL].padRight(pressurePadding));

      case HU_STOCK:
        displayText(1, tirePressure[TIRE_FL].padRight(pressurePadding) + rpmMessage + rpm.padRight(4) + F(" E:") + temperature.padRight(3) + " " + tirePressure[TIRE_FR].padRight(pressurePadding));
        displayText(2, tirePressure[TIRE_RL].padRight(pressurePadding) + spdMessage + carSpeed.padRight(4) + F(" T:") + tireTemperature.padRight(3) + " " + tirePressure[TIRE_RR].padRight(pressurePadding));

      case HU_CHINESE_WITH_CAN_SIMPLE:
        displayText(1, tirePressure[TIRE_FL].padRight(pressurePadding) + clockMessage.padCenter(textMsgLength+2) + tirePressure[TIRE_FR].padRight(pressurePadding));
        displayText(2, tirePressure[TIRE_RL].padRight(pressurePadding) + temperatureMessage.padCenter(textMsgLength+2) + tirePressure[TIRE_RR].padRight(pressurePadding));

      case HU_CHINESE_WITH_CAN_EXTENDED:
        displayText(1, tirePressure[TIRE_FL].padRight(pressurePadding) + rpmMessage + rpm.padRight(4) + F(" E:") + temperature.padRight(3) + " " + tirePressure[TIRE_FR].padRight(pressurePadding));
        displayText(2, tirePressure[TIRE_RL].padRight(pressurePadding) + spdMessage + carSpeed.padRight(4) + F(" ") + clockMessage + F(" ") + tirePressure[TIRE_RR].padRight(pressurePadding));

Great work BTW, Just what I was looking for!

bmwhitetx avatar Feb 10 '20 21:02 bmwhitetx