search

eyalcha / thermal

Thermal camera for Home Assistant
Apache License 2.0
32 stars 7 forks source link

Platform error sensor.thermal - Integration 'thermal' not found. #6

Open naamah75 opened 3 years ago

naamah75 commented 3 years ago

Since upgraded to Home Assistant core-2021.6.0 I started having this error Platform error sensor.thermal - Integration 'thermal' not found. I've made any change to my configuration, just upgrade from core-2021.5.9 to core-2021.6.0.

tjntomas commented 3 years ago

I think it has to do with the requirement to have a version entry in the manifest.json file, i.e.

{
    "domain": "thermal",  
    "name": "Thermal Vision", 
    "documentation": "", 
    "dependencies": [],  
    "codeowners": ["@eyalcha"], 
    "requirements": ["colour", "numpy", "pillow"],
    "versions": "1.0.0"
}

Try adding the version entry in the manifest.json file and see if it works.

naamah75 commented 3 years ago

Nope... It didn't work

naamah75 commented 3 years ago

The missing entry in manifest.json is "iot_class": "local_polling" Now it works.

sanderlv commented 3 years ago

Should be added via a new version via HACS...

sanderlv commented 3 years ago

FWIW, I had some trouble finding the correct syntax and this works:

(@tjntomas it's not versionsbut version:-)_

{
  "domain": "thermal",
  "name": "Thermal Vision",
  "documentation": "",
  "dependencies": [],
  "codeowners": ["@eyalcha"],
  "requirements": ["colour", "numpy", "pillow"],
  "version": "1.0.0",
  "iot_class": "local_polling"
}
TheRealWaldo commented 3 years ago

FYI @sanderlv @tjntomas and @naamah75 I forked and completely rebuilt this project at https://github.com/TheRealWaldo/thermal

It should be backward-compatible with your current setups, but I've also added some new features.

While I haven't added it to HACS yet, you can manually add the repository to HACS. However, as it currently has the same domain, you'd have to remove the old one. I may change that shortly, though.

Input and feature requests are welcome!

sanderlv commented 3 years ago

I forked my configuration.yaml and added the second integration via hacs... not sure yet how and what the differences are... Hope this camera will be ported to esphome in the (near) future...

TheRealWaldo commented 3 years ago

Hi @sanderlv. Unfortunately, esphome doesn't do images or cameras yet, so even if the AMG8833 sensor is supported, you'd still need an integration to process the data and turn it into a camera. I am working on an esphome version of the sensor integration but can't do the camera. I'll also be open-sourcing my version of the stand-alone hardware sensor soon.

Currently working on submitting my version to HACS officially, just checking off the last few boxes!