search

veista / smartthings

A fork of the Home Assistant SmartThings Integration. This adds better support for Samsung OCF Devices.
172 stars 40 forks source link

Dishwasher some values not updating #92

Open blair287 opened 11 months ago

blair287 commented 11 months ago

Got 3 devices a fridge and washing machine are working fine but the dishwasher isn't updating some of its values unless the integration is reloaded energy and switch are fine but job state etc don't update without a reload any way to fix this please?

veista commented 11 months ago

Your problems seems to be related to the smartthings subscription limit. Try splitting your devices into multiple accounts.

blair287 commented 11 months ago

Your problems seems to be related to the smartthings subscription limit. Try splitting your devices into multiple accounts.

Ah yes your right can see it in the logs.

Can I disable entities that I don't need to free up subscriptions or will it still try to subscribe to them?

veista commented 11 months ago

That is one option. Other is to split devices into multiple smartthings accounts.

blair287 commented 11 months ago

That is one option. Other is to split devices into multiple smartthings accounts.

I disabled the entities I didn't need and reloaded integration but it's still showing a failed to create subscription error in logs for the same 5 items. I disabled about 10 entities before I reloaded.

Don't want to create a second account as the devices all need to show up on fridge screen.

I assume disabling the entities didn't work as still shows 45 in integration and I have 5 that won't subscribe.

Would it be possible to point me in direction to edit the integration to delete things like delta energy and firmware upgrade required that would free up 6 subscriptions then.

Screenshot_20231216_081726_Home Assistant

veista commented 11 months ago

You need to disable them in code.

blair287 commented 11 months ago

You need to disable them in code.

Tried commenting out the firmware update in sensor.py file but it didn't work still showed 45 entities.

blair287 commented 11 months ago

You need to disable them in code.

Are you able to give me an idea where to comment out these sensors?

A good feature for your closed source would be to allow you to specifically select what subscription ls you need to prevent this issue.