-
Notifications
You must be signed in to change notification settings - Fork 23
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
Plant Daily light integral unit (mol/d·m²) changed to strange unit (mol/s·m²s) #148
Comments
Not sure what has happened there. I sill have the correct unit here. But it might be related to this, somehow: |
Don't worry about the screenshot in the README. That is very old, and I did have "s" in the first iterations, I believe. |
hm... as far as I can see, it is still in the sources:
class PlantTotalLightIntegral(IntegrationSensor):
|
PPFD is measured in Total Light Integral and Daily Light Integral is then calculated from PPFD, and those are measured in Basically Total Light Integral is the sum of
And neither of these explain why you have an extra "s" int the unit |
I think the confusion comes from the missing brackets. How to interpret mol/s⋅m²s ? I assume the system means: Or in another notation: The s⁻¹ and the s cancel each other out and this remains: And that's the correct unit for the x axis for the integral sum of PPFD. |
Not sure what I can contribute here, but I am seeing the same warnings about the DLI unit change:
|
another feeling I have so far is, that the change somehow came with 2024.4.0 of HomeAssistant... It is just for a couple of days now, that I can't see the statistic graph on the "more info" page - and need to go to the long term stats to see those. I will setup another HA instance, running 2024.2 and 2024.3 to test against these versions over the weekend. |
I have not upgraded to 2024.4 yet, but it's on my weekend plans. |
Even tho I would not understand what change it should be in HA... |
I have a hunch that HA tries to be "smart", and that since unit_time is seconds for total, it changes the unit_of_measurement from 'd' to 's' by itself |
hm... I can't imagine, tbh... Also, the Unit was always "seconds" but used the depricated TIME_SECONDS ... this was changed to the UnitOfTime.SECONDS
|
I am trying to get daily light Integral mol/d*m² or at least similar as a Default unit into ha - since the fyta Sensor ist already using a similar value it would make Sense in my opinion to unify the Units through all Sensors... / Integrations 🤔 |
This is due to home-assistant/core#115208 |
Since upgrading to homeassistant-plant Version 2024.2.0 the plant Daily light integral unit has changed to a strange unit (mol/s·m²s).
The correct unit is (mol/d·m²).
Also there are homeassistant.components.sensor.recorder errors in the log:
The text was updated successfully, but these errors were encountered: