snowzach / doods

DOODS - Dedicated Open Object Detection Service
MIT License
303 stars 31 forks source link

Label variable in DOODS intergration doco for HA #55

Closed allenrong88 closed 2 years ago

allenrong88 commented 3 years ago

HA doco for DOODS: https://www.home-assistant.io/integrations/doods/

Tried utilizing DOODS on my HA instances, worked so far without adding label variable. When I looked at the doco for label variable in order to reduce detected object to save some resources, I thought there was a bit misleading on the following highlighted area.

image

I was expecting the following code in stead of the one shown in the sample configuration. Would you be able to confirm it please?

If that's the case, would you be able to update the doco to reduce confusion in the future?

Thx so much for this amazing project. Love it!

snowzach commented 3 years ago

I will try to update that in the future. Their documentation system doesn't allow you to specify the the formats that doods will take. this is basically the format that tensorflow takes also.

allenrong88 commented 3 years ago

Thx for for your reply, really nice integration.

So I just would like to get my head clear about the correct format of the code I will be using in my HA. I will put the below code into my config, is that correct?

labels:

Thank you again

snowzach commented 3 years ago

That should actually work fine...

allenrong88 commented 3 years ago

Thx Zach, and I will put my code above into HA config to see how it goes.