Olen / homeassistant-plant

Alternative Plant component of home assistant
365 stars 25 forks source link

Feature request: Allow changing PPFD and DLI to other sensors #190

Open TheWarped opened 3 months ago

TheWarped commented 3 months ago

The diagnostic variables "ppfd (mol)" and "Total ppfd (mol) Integral" cannot be changed to other sensors like the thresholds from the configuration.
While the DLI and PPFD calculations seem to work well, I'd already made PPFD and DLI template sensors before finding this integration and prefer to use those as I've calibrated them with an Apogee PAR meter.

Olen commented 3 months ago

Sounds like a nice idea. It would, however, need quite big changes to the code base. As I don't have any such sensors myself, I depend on others to create and test a PR to implement this.

TheWarped commented 3 months ago

Understandable. I'll see about contributing something for that then.

jeffcybulski commented 2 months ago

Is this why replacing the ppfd entity with a Hass-Variables irradiance sensor has not affect on the ppfd and dli calculation? It seems to be deriving it from lux (for some reason and somehow...). If I change lux to ppfd assuming white LED lights and a scaling factor of 1.85×10^−8, you arrive at the right PAR

Olen commented 2 months ago

Ppfd and dli are derived from lx. They can currently not be replaced.

PRs are welcome