homeassistant-powercalc icon indicating copy to clipboard operation
homeassistant-powercalc copied to clipboard

Add support for Philips Hue 548503 BR30

Open davidski opened this issue 1 year ago • 5 comments

This adds support for Philips Hue 548503 Smart Light BR30. I'm new to this project (and HASS itself) so any feedback/corrections are greatfully accepted. Thanks for the nifty integration!

davidski avatar Aug 21 '22 10:08 davidski

JSON Schema validation failed for custom_components/powercalc/data/signify/548727/model.json


Validator: minimum Validator value:

0.05

Message:

'0 is less than the minimum of 0.05'

Instance:

0

github-actions[bot] avatar Aug 21 '22 10:08 github-actions[bot]

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarcloud[bot] avatar Aug 21 '22 10:08 sonarcloud[bot]

JSON Schema validation failed for custom_components/powercalc/data/signify/548727/model.json

Validator: minimum Validator value:

0.05

Message:

'0 is less than the minimum of 0.05'

Instance:

0

Darn. I looked over the color_temp file, but missed that zero in standby power in the model.json. Any tips on how to fix/adjust that? Some sort of dummy load + remeasure, perhaps (I'm unsure how to set that up).

davidski avatar Aug 21 '22 10:08 davidski

@davidski Thanks for the contribution!

There seem to be a lack of resolution with your powermeter. Especially noticable on the lower watt values. It goes from 1.2 to 0.6 to 0.9. It could fluctuate a littlebit, but not this much. There seem to be some pattern with gaps of 0.3 watt. Do you happen to own another smartplug which you could use, which hopefully is more accurate and has better precision. At least 0.1 watt precision is needed to get a good quality LUT file.

When you are unable to measure the low watts (especially the standby power) than you can have a look into attaching multiple of the same bulb model (but this will only be possible if you own multiple). Or you can attach a dummy load (you can only use fully restive loads for that, for example Incandescent bulb or a resistor, see this topic https://github.com/bramstroker/homeassistant-powercalc/issues/371#issuecomment-1141437437)

Some additional notes about the PR. You can omit the supported_models next time as this file will be automatically modified after the PR is merged by a github action.

Could you have a look into the device page of the light in HA and note the manufacturer and model listed there? Than we can also make sure auto discovery will work correctly for this bulb

Name in model.json needs to be the advertised name, so you can put "Hue White and Color Ambiance BR30" there.

Btw nice light bulb, didn't know it existed.

bramstroker avatar Aug 21 '22 12:08 bramstroker

Drat. I was afraid about the sensitivity issue (Sengled Smart Plugs for anyone that comes across this). Thanks for all the feedback. I do have some CT sensors on branch circuits which are probably more sensitive. That just requires some rejiggering things to find an isolated branch to test this on. I'll work on that, changing this PR to a draft until that's done.

davidski avatar Aug 21 '22 13:08 davidski

@davidski Do you still have plans to work on this? When this is not the case I will close the PR.

bramstroker avatar Sep 23 '22 14:09 bramstroker

Hi! Sorry for the delay on this. My second attempt at getting better resolution failed. I've got some Shelly PM relays that I'm going to try as well, but it's going to take me a bit to get to wiring that up and retesting. If you want to close the PR, I certainly won't mind, but I do have hopes that this third try will allow good metrics to be gathered. 🤞

davidski avatar Sep 23 '22 14:09 davidski

Thanks @davidski. No need to hurry. I was just going through all open issues and PR's to clean up a bit. This PR can stay open, no problem. Hope you'll get good figures with the Shelly's.

bramstroker avatar Sep 23 '22 15:09 bramstroker