{"action":"created","issue":{"url":"https://api.github.com/repos/home-assistant/home-assistant/issues/18129","repository_url":"https://api.github.com/repos/home-assistant/home-assistant","labels_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/18129/labels{/name}","comments_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/18129/comments","events_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/18129/events","html_url":"https://github.com/home-assistant/home-assistant/pull/18129","id":376933881,"node_id":"MDExOlB1bGxSZXF1ZXN0MjI4MDMyNzQ4","number":18129,"title":"Fix DTE Energy Bridge V2 scaling issue. (#18124)","user":{"login":"mtl010957","id":7416838,"node_id":"MDQ6VXNlcjc0MTY4Mzg=","avatar_url":"https://avatars2.githubusercontent.com/u/7416838?v=4","gravatar_id":"","url":"https://api.github.com/users/mtl010957","html_url":"https://github.com/mtl010957","followers_url":"https://api.github.com/users/mtl010957/followers","following_url":"https://api.github.com/users/mtl010957/following{/other_user}","gists_url":"https://api.github.com/users/mtl010957/gists{/gist_id}","starred_url":"https://api.github.com/users/mtl010957/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mtl010957/subscriptions","organizations_url":"https://api.github.com/users/mtl010957/orgs","repos_url":"https://api.github.com/users/mtl010957/repos","events_url":"https://api.github.com/users/mtl010957/events{/privacy}","received_events_url":"https://api.github.com/users/mtl010957/received_events","type":"User","site_admin":false},"labels":[{"id":528349273,"node_id":"MDU6TGFiZWw1MjgzNDkyNzM=","url":"https://api.github.com/repos/home-assistant/home-assistant/labels/cla-error","name":"cla-error","color":"ff0000","default":false},{"id":527909495,"node_id":"MDU6TGFiZWw1Mjc5MDk0OTU=","url":"https://api.github.com/repos/home-assistant/home-assistant/labels/cla-needed","name":"cla-needed","color":"ffff00","default":false},{"id":955422305,"node_id":"MDU6TGFiZWw5NTU0MjIzMDU=","url":"https://api.github.com/repos/home-assistant/home-assistant/labels/in%20progress","name":"in progress","color":"ededed","default":false},{"id":620177376,"node_id":"MDU6TGFiZWw2MjAxNzczNzY=","url":"https://api.github.com/repos/home-assistant/home-assistant/labels/platform:%20sensor.dte_energy_bridge","name":"platform: sensor.dte_energy_bridge","color":"ededed","default":false},{"id":832964196,"node_id":"MDU6TGFiZWw4MzI5NjQxOTY=","url":"https://api.github.com/repos/home-assistant/home-assistant/labels/small-pr","name":"small-pr","color":"e5bd7e","default":false}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2018-11-02T19:08:11Z","updated_at":"2018-11-02T19:45:05Z","closed_at":null,"author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/18129","html_url":"https://github.com/home-assistant/home-assistant/pull/18129","diff_url":"https://github.com/home-assistant/home-assistant/pull/18129.diff","patch_url":"https://github.com/home-assistant/home-assistant/pull/18129.patch"},"body":"## Description:\r\nThis fixes the scaling issue with the DTE Energy Bridge version 2 device. The existing code included a section to fix the scaling that worked around a bug with the version 1 device. The version 2 device does not exhibit the same behaviour. As far as I've been able to test the version 2 device always returns instantaneous energy usage values in the form \"nnnnnn.nnn kW\". The restful interface from the device is undocumented. The values are actually in Watts, even though the string includes the \"kW\" characters. In the case of the version 2 device the value returned will always be divided by 1000 to make kiloWatts.\r\n\r\n**Related issue (if applicable):** fixes #18124\r\n\r\n**Pull request in [home-assistant.io](https://github.com/home-assistant/home-assistant.io) with documentation (if applicable):** no change required\r\n\r\n## Example entry for `configuration.yaml` (if applicable):\r\n```yaml\r\n\r\n```\r\n\r\n## Checklist:\r\n  - [X] The code change is tested and works locally.\r\n  - [X] Local tests pass with `tox`. **Your PR cannot be merged unless tests pass**\r\n\r\nIf user exposed functionality or configuration variables are added/changed:\r\n  - [ ] Documentation added/updated in [home-assistant.io](https://github.com/home-assistant/home-assistant.io)\r\n\r\nIf the code communicates with devices, web services, or third-party tools:\r\n  - [ ] New dependencies have been added to the `REQUIREMENTS` variable ([example][ex-requir]).\r\n  - [ ] New dependencies are only imported inside functions that use them ([example][ex-import]).\r\n  - [ ] New or updated dependencies have been added to `requirements_all.txt` by running `script/gen_requirements_all.py`.\r\n  - [ ] New files were added to `.coveragerc`.\r\n\r\nIf the code does not interact with devices:\r\n  - [ ] Tests have been added to verify that the new code works.\r\n\r\n[ex-requir]: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/keyboard.py#L14\r\n[ex-import]: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/keyboard.py#L54\r\n"},"comment":{"url":"https://api.github.com/repos/home-assistant/home-assistant/issues/comments/435488344","html_url":"https://github.com/home-assistant/home-assistant/pull/18129#issuecomment-435488344","issue_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/18129","id":435488344,"node_id":"MDEyOklzc3VlQ29tbWVudDQzNTQ4ODM0NA==","user":{"login":"homeassistant","id":23041403,"node_id":"MDQ6VXNlcjIzMDQxNDAz","avatar_url":"https://avatars3.githubusercontent.com/u/23041403?v=4","gravatar_id":"","url":"https://api.github.com/users/homeassistant","html_url":"https://github.com/homeassistant","followers_url":"https://api.github.com/users/homeassistant/followers","following_url":"https://api.github.com/users/homeassistant/following{/other_user}","gists_url":"https://api.github.com/users/homeassistant/gists{/gist_id}","starred_url":"https://api.github.com/users/homeassistant/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/homeassistant/subscriptions","organizations_url":"https://api.github.com/users/homeassistant/orgs","repos_url":"https://api.github.com/users/homeassistant/repos","events_url":"https://api.github.com/users/homeassistant/events{/privacy}","received_events_url":"https://api.github.com/users/homeassistant/received_events","type":"User","site_admin":false},"created_at":"2018-11-02T19:45:05Z","updated_at":"2018-11-02T19:45:05Z","author_association":"NONE","body":"Hi @mtl010957, \n\nIt seems you haven't yet signed a CLA. Please do so [here](https://home-assistant.io/developers/cla_sign_start/?pr=home-assistant&#x2F;home-assistant#18129).\n\nOnce you do that we will be able to review and accept this pull request.\n\nThanks!\n"}}
{"actor":{"display_login":"homeassistant"}}