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

Battery temperature exactly double the temperature reported in the Fox cloud 2.0 app #284

Open
Tom8260 opened this issue Jan 14, 2025 · 2 comments
Labels
bug Something isn't working

Comments

@Tom8260
Copy link

Tom8260 commented Jan 14, 2025

Describe the bug
A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  1. add a new card to home assistant
  2. Add a history Graph
  3. Add the FoxESS - Bat Temperature
  4. click save

Expected behavior
It should show the battery temperature. If I compare this with the temperature shown in the Andriod Fox Cloud 2 app it is twice that value. The Fox Cloud 2 app is showing the correct temperature.
Screenshots
image

Debug logs:

**HA version:
Core
2025.1.2
Supervisor
2024.12.3
Operating System
14.1
Frontend
20250109.0

**foxess-ha version: 0.4

@Tom8260 Tom8260 added the bug Something isn't working label Jan 14, 2025
@Daibutt
Copy link
Collaborator

Daibutt commented Jan 14, 2025

I think this is due to legacy Fox naming convention rather than a bug.
Bat Temperature is the BMS temp not the battery. In the FoxCloud website you can see BatTemp and this will liley match what you are seeing in HA.
In Fox app V2, they chnaged it to show the lowest cell temp from across all your batteries which is way more useful.
I don't believe the equivalent (cell temp low) is available with this HA Fox Cloud integration, although it is if you ever move to the HA Fox Modbus integration.
There are also some oddities with the naming of ambient temp and inv temp, explained here;
https://github.com/TonyM1958/HA-FoxESS-Modbus/wiki/Temperatures

@FozzieUK
Copy link
Collaborator

Spot on Dave ☺️, the only thing to add is Fox have been asked and confirmed they will add the lowest cell temperature to the OpenAPI, as soon as they do I will add it here (and Al will add to Energy Stats)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants