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.

Sign up for GitHub

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

Jump to bottom

[Bug]: Home Assistant error caused by undefined LDR values #111

Closed
bcameron21 opened this issue Apr 30, 2023 · 3 comments · Fixed by #106 or sblantipodi/glow_worm_luciferin#49
Closed

[Bug]: Home Assistant error caused by undefined LDR values #111

bcameron21 opened this issue Apr 30, 2023 · 3 comments · Fixed by #106 or sblantipodi/glow_worm_luciferin#49
Assignees
@sblantipodi
Labels
bug Something isn't working keep don't stale this issue

Comments

@bcameron21
Copy link

Firefly Luciferin version

2.10.10

Glow Worm Luciferin version

5.9.7

Firmware type

FULL

What is the stream method?

MQTT Stream

Fiefly Luciferin config file

No response

Relevant log output

No response

How to reproduce

My error logs are filled with warnings:

Template variable warning: 'dict object' has no attribute 'ldr' when rendering '{{ value_json.ldr }}'

I don't have a light density resistor installed or active.
It seems that the Esp devices is passing a setting, but Home Assistant doesn't know what to do with it.

In Home Assistant it is picking up a Luciferin LDR entity from MQTT, with a value "Unknown".

Using MQTTExplorer I can see a config paramter being passed in the topic
luciferin/ldr/config
with the follwing value template:

{
"name": "Luciferin LDR",
"icon": "mdi:theme-light-dark",
"unique_id": "Luciferin_LDR",
"state_topic": "lights/glowwormluciferin",
"value_template": "{{ value_json.ldr }}",
"unit_of_measurement": "%"
}
@bcameron21 bcameron21 added the bug Something isn't working label Apr 30, 2023
@sblantipodi
Copy link
Owner

hi @bcameron21,
did you enabled the HA integration?

https://github.com/sblantipodi/firefly_luciferin/wiki/Home-Automation-configs

@bcameron21
Copy link
Author

Good point. Rereading that link, as well as a conversation on Discord I realised that I was using the package version of the integration. Removed that from HA, and let Luciferin set itself up with MQTT autodiscovery.

After this I was still getting the error in the logs.

Using MQTT Explorer I deleted the retained config message mentioned above, and that stops the error message.

However if I reset the config by clicking on the tick next to the MQTT discovery topic in the Luciferin settings, then that adds it back to MQTT. This also recreates the entity in HA (i.e. sensor.luciferin_ldr). And the errors start again!

The workaround I'm going with for now is to delete the MQTT retained message.

Not sure how else to investigate this further.

@sblantipodi
Copy link
Owner

Hi @bcameron21,
ok I identified the problem and fixed it.

You have found a workaround already, next release is going to fix the problem once and for all :)

Thank you for reporting it, this is much appreciated!!!

@sblantipodi sblantipodi added the keep don't stale this issue label May 1, 2023
sblantipodi added a commit that referenced this issue May 1, 2023
@sblantipodi
Closes #111
0dbd8ef
@sblantipodi sblantipodi mentioned this issue May 1, 2023
Merged
@sblantipodi sblantipodi mentioned this issue May 9, 2023
Merged
sblantipodi added a commit that referenced this issue May 13, 2023
@sblantipodi
Smoothing with frame insertion. (#106)
2a848b0 
- ***Breaking changes***: requires `Glow Worm Luciferin` (v5.10.6)  
- **Added a [smoothing feature that is used to smooth the transitions from one color to another](https://github.com/sblantipodi/firefly_luciferin/wiki/Smoothing-color-transitions)**,
this is particularly useful to reduce eye strain with contents that produces fast flashing like fast peaced games or similar. This setting can be controlled on the fly via [MQTT/Home Assistant](https://github.com/sblantipodi/firefly_luciferin/wiki/Home-Automation-configs). Closes [#80](#80) and [#109](#109). 
- **Added a [latency test](https://github.com/sblantipodi/firefly_luciferin/wiki/(DRAFT-this-feature-will-be-available-on-the-next-release)-Test-image-and-latency-test)**. This test displays colors in rapid succession and helps you check if the latency between the image shown on your monitor and the color displayed on the led strip is acceptable to you. Highering the framerate helps reducing the latency. You can run this test at 10 different speed. This is also useful when choosing the right smoothing level for your preferences.
- **[Web Interface](https://github.com/sblantipodi/firefly_luciferin/wiki/Remote-Access#luciferin-web-interface) now displays an Auto save button**, if auto save is enabled, color and brightness information is stored into memory to retain this settings after reboot. Closes [#38](sblantipodi/glow_worm_luciferin#38).
- Added an option within the [Web Interface](https://github.com/sblantipodi/firefly_luciferin/wiki/Remote-Access#luciferin-web-interface) to choose to **turn on the LEDs once the microcontroller boots up**.
- **[Device reset](https://github.com/sblantipodi/firefly_luciferin/wiki/(DRAFT-this-feature-will-be-available-on-the-next-release)-Device-reset)** has been improved. Closes [#100](#100) and [#37](sblantipodi/glow_worm_luciferin#37). 
- **Added support for [RGB and BGR color order](https://github.com/sblantipodi/firefly_luciferin/wiki/RGB-and-RGBW-support)**. 
- **RGBW SK6812 performance boost**. +30% maximum framerate and reduced latency.
- **[Added support for the QuinLED dig2go](https://github.com/sblantipodi/firefly_luciferin/wiki/Compatible-Hardware#pre-build-boards-support) pre-build board**.
- Added support for **[Hardware Button](https://github.com/sblantipodi/firefly_luciferin/wiki/Hardware-Button)**.
- Fixed an issue that prevented correct screen capture once changed the desired framerate or aspect ratio.
- Minor UI improvements.
- [Web Installer](https://sblantipodi.github.io/glow_worm_luciferin) now presents an option to install beta or previous version of the Glow Worm Luciferin firmware.
- Improved [log level configurability](https://github.com/sblantipodi/firefly_luciferin/wiki/Debug).
- Removed warnings in the Home Assistant logs. Closes [#111](#111).
- [Arduino Bootstrapper](https://github.com/sblantipodi/arduino_bootstrapper/releases) update (v.1.14).
sblantipodi added a commit to sblantipodi/glow_worm_luciferin that referenced this issue May 13, 2023
@sblantipodi
Smoothing with frame insertion. (#49)
6c3184d 
- ***Breaking changes***: requires `Firefly Luciferin` (v2.11.7)
- **Added a [smoothing feature that is used to smooth the transitions from one color to another](https://github.com/sblantipodi/firefly_luciferin/wiki/Smoothing-color-transitions)**,
this is particularly useful to reduce eye strain with contents that produces fast flashing like fast peaced games or similar. This setting can be controlled on the fly via [MQTT/Home Assistant](https://github.com/sblantipodi/firefly_luciferin/wiki/Home-Automation-configs). Closes [#80](sblantipodi/firefly_luciferin#80) and [#109](sblantipodi/firefly_luciferin#109). 
- **Added a [latency test](https://github.com/sblantipodi/firefly_luciferin/wiki/Test-image-and-latency-test)**. This test displays colors in rapid succession and helps you check if the latency between the image shown on your monitor and the color displayed on the led strip is acceptable to you. Highering the framerate helps reducing the latency. You can run this test at 10 different speed. This is also useful when choosing the right smoothing level for your preferences.
- **[Web Interface](https://github.com/sblantipodi/firefly_luciferin/wiki/Remote-Access#luciferin-web-interface) now displays an Auto save button**, if auto save is enabled, color and brightness information is stored into memory to retain this settings after reboot. Closes [#38](#38).
- Added an option within the [Web Interface](https://github.com/sblantipodi/firefly_luciferin/wiki/Remote-Access#luciferin-web-interface) to choose to **turn on the LEDs once the microcontroller boots up**.
- **[Device reset](https://github.com/sblantipodi/firefly_luciferin/wiki/Device-reset)** has been improved. Closes [#100](sblantipodi/firefly_luciferin#100) and [#37](#37). 
- **Added support for [RGB and BGR color order](https://github.com/sblantipodi/firefly_luciferin/wiki/RGB-and-RGBW-support)**. 
- **RGBW SK6812 performance boost**. +30% maximum framerate and reduced latency.
- **[Added support for the QuinLED dig2go](https://github.com/sblantipodi/firefly_luciferin/wiki/Compatible-Hardware#pre-build-boards-support) pre-build board**.
- Added support for **[Hardware Button](https://github.com/sblantipodi/firefly_luciferin/wiki/Hardware-Button)**.
- Fixed an issue that prevented correct screen capture once changed the desired framerate or aspect ratio.
- Minor UI improvements.
- [Web Installer](https://sblantipodi.github.io/glow_worm_luciferin) now presents an option to install beta or previous version of the Glow Worm Luciferin firmware.
- Improved [log level configurability](https://github.com/sblantipodi/firefly_luciferin/wiki/Debug).
- Removed warnings in the Home Assistant logs. Closes [#111](sblantipodi/firefly_luciferin#111).
- [Arduino Bootstrapper](https://github.com/sblantipodi/arduino_bootstrapper/releases) update (v.1.14).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sblantipodi sblantipodi

Labels
bug Something isn't working keep don't stale this issue
Projects
None yet
Milestone
No milestone
2 participants
@sblantipodi @bcameron21