Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fisher & Paykel oven not showing celsius correctly #248

Open
kaygang opened this issue Apr 23, 2024 · 5 comments
Open

Fisher & Paykel oven not showing celsius correctly #248

kaygang opened this issue Apr 23, 2024 · 5 comments

Comments

@kaygang
Copy link

kaygang commented Apr 23, 2024

Hi there, I have added my Fisher & Paykel smart oven via the integration and it's working as expected. However, the temperature numbers on the Home Assistant control card show as Fahrenheit even though it is using the Celsius marker/format. I need the numbers in Celsius basically. Is there any way to fix this? Oddly enough, the Raw Temperature sensor shows the correct temperature in Celsius!

image image
@simbaja
Copy link
Owner

simbaja commented May 4, 2024

It might be that the oven always returns measurements in Fahrenheit (similar to other devices). If you modify the temperature_unit property in the ge_oven.py file to comment out the if logic (i.e. always return UnitOfTemperature.Fahrenheit), does it all still work for you (including setting cook temp and mode)?

@kaygang
Copy link
Author

kaygang commented May 4, 2024

Thanks heaps for the reply, this is what I changed to in that file, but it didn't seem to make any difference.

@Property
def temperature_unit(self):
measurement_system = self.appliance.get_erd_value(ErdCode.TEMPERATURE_UNIT)
if measurement_system == ErdMeasurementUnits.METRIC:
return UnitOfTemperature.CELSIUS
else:
return UnitOfTemperature.CELSIUS
# return UnitOfTemperature.FAHRENHEIT

@simbaja
Copy link
Owner

simbaja commented May 8, 2024

If you comment out everything other than the last line (i.e. reverse what you've done above), does it all work correctly?

@parndt
Copy link

parndt commented May 21, 2024

@simbaja that workaround works well for me for setting/reading current temperature as I have the same issue with ULV316452

	@property
	def temperature_unit(self):
		# measurement_system = self.appliance.get_erd_value(ErdCode.TEMPERATURE_UNIT)
		# if measurement_system == ErdMeasurementUnits.METRIC:
		#		return TEMP_CELSIUS
		return TEMP_FAHRENHEIT

However, the other metrics are suffering conversion as you can see in the screenshot:

image

@8jmm3
Copy link

8jmm3 commented Jan 17, 2025

I have the same issue with a GE Cafe oven and the temperature set to Celsius in Home Assistant. I will also attempt the workaround.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants