adonno / tagreader

Simple to build/use NFC tag reader, specially created for Home Assistant
https://adonno.com
GNU General Public License v3.0
1.13k stars 175 forks source link

ESPHome 2023.7 - Error at firmware update #245

Closed ghost closed 8 months ago

ghost commented 11 months ago

With ESPHome 2023.7 the following error occurs. It seems, the configuration in your repository needs to be changed.

INFO ESPHome 2023.7.0 
INFO Reading configuration /config/esphome/tagreader-7c1293.yaml... 

Failed config 

switch.template: [source /config/esphome/.esphome/packages/fa8076cc/tagreader.yaml:56] 
platform: 
template name: TagReader Buzzer Enabled 
id: buzzer_enabled icon: 
mdi:volume-high 
optimistic: True 
The restore_state option has been removed in 2023.7.0. Use the restore_mode option instead.
restore_state: True 
entity_category: config 

switch.template: [source /config/esphome/.esphome/packages/fa8076cc/tagreader.yaml:63] 
platform: template 
name: TagReader LED enabled 
id: led_enabled icon: mdi:alarm-light-outline 
optimistic: True 
The restore_state option has been removed in 2023.7.0. Use the restore_mode option instead.
restore_state: True 
entity_category: config
Masterz69 commented 11 months ago

Reason is:

https://github.com/esphome/esphome/pull/5106 This removes restore_state on a template switch which was replaced with restore_mode on the switch core config and having both is just confusing.

howesey2k commented 11 months ago

So what do we need to do to update it?

Many thanks.

Masterz69 commented 11 months ago

Remove restore_state from template switches. Specify restore_mode if not defined yet.

jakevis commented 11 months ago

yeah running into this issue as well - cant work out how to override the template - but I guess ideally this needs to be updated in https://github.com/adonno/tagreader/blob/master/tagreader.yaml

howesey2k commented 11 months ago

I agree for it to be looked at. I've deleted the references to restore_state and reinstalled it. Seems to be working but getting an error "Component pn532 took a long time for an operation" Could this be looked at? Many thanks

renewoensdregt commented 11 months ago

there is already a pull request to update this. Once approved it should be fixed. https://github.com/adonno/tagreader/pull/247

monstergubl commented 11 months ago

hey got the same problem did you get a solution ?

howesey2k commented 11 months ago

Hi, the firmware issue has been resolved. There has been an update in the yaml. I just copied and pasted and it's working again.

waazdakka commented 10 months ago

Hello, I just flashed again my tagreader with the actual yaml but I still get the "Component pn532 took a long time for an operation (0.10 s). [W][component:205]: Components should block for at most 20-30ms." error. Don't know what to change to make this error disappear :'(

howesey2k commented 10 months ago

I get the same error. I think it's still working with this error but please check too. I think someone mentioned in a forum post that it's possibly an ESPHome issue. Might be worth mentioning it on their forums. The firmware update fixed a different issue which stopped the device from working.

On Sat, 19 Aug 2023, 12:49 waazdakka, @.***> wrote:

Hello, I just flashed again my tagreader with the actual yaml but I still get the "Component pn532 took a long time for an operation (0.10 s). [W][component:205]: Components should block for at most 20-30ms." error. Don't know what to change to make this error disappear :'(

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1684926111, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZPFDWKVQZ66FB7OQL3XWCR4DANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>

titis14 commented 10 months ago

Hi, I still get the error even though I have the latest version.

howesey2k commented 10 months ago

What error you getting exactly?

On Sat, 26 Aug 2023, 21:26 titis14, @.***> wrote:

Hi, I still get the error even though I have the latest version.

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1694495865, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZKBSLBDJWC6TLD7JG3XXJLYDANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>

titis14 commented 10 months ago

This mistake. [11:50:41][W][component:204]: Component pn532 took a long time for an operation (0.10 s). [11:50:41][W][component:205]: Components should block for at most 20-30ms. For a while now my tagreader has not been able to read badges. I made a topic here but no one replied. https://github.com/adonno/tagreader/issues/241 I don't know if it's related to this error. I have this project and for several months at my club people can no longer deactivate the alarm easily. Thanks

howesey2k commented 10 months ago

Did you copy and paste the yaml update that adonno released? This fixed my tags from reading. You need to copy and the paste the new yaml into the correct file. Then click install again. This won't fix the timeout issue in the logs but should fix the tags from not scanning. If this doesn't work, try reaching out again and see if someone else can help you like a developer :) If you are unsure where to copy and paste the yaml, I'll see if I can remember haha

On Sun, 27 Aug 2023, 10:53 titis14, @.***> wrote:

This mistake. [11:50:41][W][component:204]: Component pn532 took a long time for an operation (0.10 s). [11:50:41][W][component:205]: Components should block for at most 20-30ms. For a while now my tagreader has not been able to read badges. I made a topic here but no one replied.

241 https://github.com/adonno/tagreader/issues/241

I don't know if it's related to this error. I have this project and for several months at my club people can no longer deactivate the alarm easily. Thanks

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1694623247, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZPX72FSOZI6NCB7F3LXXMKLBANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>

titis14 commented 10 months ago

On ESPHome I clicked on edited and I put the new yaml then installed by wifi.

howesey2k commented 10 months ago

It's actually in a folder on your machine you need to navigate to. Do you use a file editor plug in?

On Sun, 27 Aug 2023, 15:41 titis14, @.***> wrote:

On ESPHome I clicked on edited and I put the new yaml then installed by wifi.

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1694685715, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZN37Q7JG57ZUISTWEDXXNMBJANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>

titis14 commented 10 months ago

No, I didn't do it through a file editor. On Home Assistant I use Studio Core Server, would you tell me which file to modify? Thanks

EDIT: I found the file but I don't understand why I only have 15 lines of the yaml file, I had to have all the ESP demon code.

howesey2k commented 10 months ago

I navigated to the yaml file by esphome/.esphome/packages/name of tag reader package/Tagreader.yaml

I then copied Adonos code and pasted it into that yaml then clicked install on the esphome integration

On Sun, 27 Aug 2023, 17:11 titis14, @.***> wrote:

No, I didn't do it through a file editor. On Home Assistant I use Studio Core Server, would you tell me which file to modify? Thanks

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1694704944, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZMES4LYAT4LGYE3MC3XXNWTZANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>

titis14 commented 10 months ago

Thank you in this way I am well in v1.4 :(

howesey2k commented 10 months ago

Did this work?

On Sun, 27 Aug 2023, 17:16 titis14, @.***> wrote:

Thank you in this way I am well in v1.4 :(

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1694706001, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZOZGWTDFVDY6X2CTYLXXNXHBANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>

titis14 commented 10 months ago

I think I know why it doesn't work anymore. Indeed I had to use the smart home junkie file. dashboard_import: package_import_url: github://smarthomejunkie/tagreader/tagreader.yaml In my editor I see that the file is not up to date. I modified the file but when I want to validate it in ESPHome I have errors. I also have 2 folder for the tag. Is there a way to erase everything and start from scratch? I do all this remotely because I don't have the ESP near me.

howesey2k commented 10 months ago

I was once given this link https://adonno.github.io/tagreader

Plug it into the usb on your pc and use edge, chrome has never worked for me.

If this doesn't work, not sure how to help sorry.

titis14 commented 10 months ago

Ok thanks you have already put on a track. I'm going to do a new ESP and we'll see.

howesey2k commented 10 months ago

Any joy?

On Sun, 27 Aug 2023, 17:45 titis14, @.***> wrote:

Ok thanks you have already put on a track. I'm going to do a new ESP and we'll see.

— Reply to this email directly, view it on GitHub https://github.com/adonno/tagreader/issues/245#issuecomment-1694711523, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASFRLZO6YVHEQOVK7FZKP6TXXN2RZANCNFSM6AAAAAA2PPPEGE . You are receiving this because you commented.Message ID: @.***>