search

tedsalmon / BlueBus

A Bluetooth module for vehicles equipped with I-Bus
Other
169 stars 41 forks source link

Temperature on BMBT UI #84

Closed nabucho closed 3 years ago

nabucho commented 3 years ago

Ambient and Oil Temperature on BMBT Support for Fahrenheit, auto configuration of unit based on cluster settings

Todo: proper localization for non-EN eventual reorder localization texts test the OIL setting on non E46 car

tedsalmon commented 3 years ago

Thanks a bunch @nabucho ! Something to note is that oil temperature has been disabled because it's only available on non-M E38, E39, E52 and E53s (anything pre-LSZ). This is because the LCM in those cases is also the CCM, so it has the oil level sensor wired to it. E46s and M5s because the IKE has the sensor wired to it for display on the cluster.

I may also tweak your new events back to a "Value Update" event to save memory.

Thanks again! -Ted

nabucho commented 3 years ago

Yes please, cleanup and fix to fit to the code best. I an aware E46 does not have the oil temp, so I disabled the menu option in that case, but was not sure about the code there, looked like it can work on other cars.