{"action":"created","comment":{"url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/comments/169643569","pull_request_review_id":98201379,"id":169643569,"diff_hunk":"@@ -0,0 +1,72 @@\n+\"\"\"Class to hold all light accesories.\"\"\"\n+import logging\n+\n+from homeassistant.const import STATE_UNKNOWN\n+from homeassistant.helpers.event import async_track_state_change\n+\n+from . import TYPES\n+from .accessories import HomeAccessory\n+from .const import (\n+    SERVICES_LIGHTBULB, CHAR_BRIGHTNESS, CHAR_STATE)\n+\n+\n+_LOGGER = logging.getLogger(__name__)\n+\n+\n+@TYPES.register('Lightbulb')\n+class Lightbulb(HomeAccessory):\n+    \"\"\"Generate a Light accessory for a lightbulb.\"\"\"\n+\n+    def __init__(self, hass, entity_id, display_name):\n+        _LOGGER.debug(\"Loading light component1\")\n+        \"\"\"Initialize a LightBulb accessory object.\"\"\"\n+        super().__init__(display_name)\n+        self.set_category(self.ALL_CATEGORIES.LIGHTBULB)\n+        _LOGGER.debug(\"Loading light component2\")\n+        self.set_accessory_info(entity_id)\n+        _LOGGER.debug(\"Loading light component3\")\n+        self.add_preload_service(SERVICES_LIGHTBULB)\n+        _LOGGER.debug(\"Loading light component4\")\n+\n+        self._hass = hass\n+        self._entity_id = entity_id\n+\n+        self.brightness = None\n+\n+        _LOGGER.debug(\"Loading light component5\")\n+\n+        self.service_lightbulb = self.get_service(SERVICES_LIGHTBULB)\n+        _LOGGER.debug(\"Loading light component6\")\n+        #self.char_brightness = self.service_lightbulb. \\\n+        #    get_characteristic(CHAR_BRIGHTNESS)\n+        _LOGGER.debug(\"Loading light component7\")\n+        self.char_state = self.service_lightbulb. \\\n+            get_characteristic(CHAR_STATE)\n+        _LOGGER.debug(\"Loading light component8\")\n+\n+        #self.char_brightness.setter_callback = self.set_brightness\n+        _LOGGER.debug(\"Loading light component9\")\n+        self.char_state.setter_callback = self.set_state\n+        _LOGGER.debug(\"Loading light component10\")\n+\n+    def run(self):\n+        \"\"\"Method called be object after driver is started.\"\"\"\n+        state = self._hass.states.get(self._entity_id)\n+        self.update_state(new_state=state)\n+\n+        async_track_state_change(\n+            self._hass, self._entity_id, self.update_state)\n+\n+    def set_brightness(self, value):\n+        pass\n+\n+    def update_brightness(self, entity_id=None, old_state=None,\n+                              new_state=None):","path":"homeassistant/components/homekit/lights.py","position":64,"original_position":64,"commit_id":"e9dfe0915b1e3d8b286a05203a52f0ed8640dffc","original_commit_id":"e9dfe0915b1e3d8b286a05203a52f0ed8640dffc","user":{"login":"houndci-bot","id":6697940,"avatar_url":"https://avatars0.githubusercontent.com/u/6697940?v=4","gravatar_id":"","url":"https://api.github.com/users/houndci-bot","html_url":"https://github.com/houndci-bot","followers_url":"https://api.github.com/users/houndci-bot/followers","following_url":"https://api.github.com/users/houndci-bot/following{/other_user}","gists_url":"https://api.github.com/users/houndci-bot/gists{/gist_id}","starred_url":"https://api.github.com/users/houndci-bot/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/houndci-bot/subscriptions","organizations_url":"https://api.github.com/users/houndci-bot/orgs","repos_url":"https://api.github.com/users/houndci-bot/repos","events_url":"https://api.github.com/users/houndci-bot/events{/privacy}","received_events_url":"https://api.github.com/users/houndci-bot/received_events","type":"User","site_admin":false},"body":"continuation line over-indented for visual indent","created_at":"2018-02-21T13:58:55Z","updated_at":"2018-02-21T13:58:56Z","html_url":"https://github.com/home-assistant/home-assistant/pull/12576#discussion_r169643569","pull_request_url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/home-assistant/home-assistant/pulls/comments/169643569"},"html":{"href":"https://github.com/home-assistant/home-assistant/pull/12576#discussion_r169643569"},"pull_request":{"href":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576"}}},"pull_request":{"url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576","id":170477198,"html_url":"https://github.com/home-assistant/home-assistant/pull/12576","diff_url":"https://github.com/home-assistant/home-assistant/pull/12576.diff","patch_url":"https://github.com/home-assistant/home-assistant/pull/12576.patch","issue_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/12576","number":12576,"state":"open","locked":false,"title":"[WIP] Initial attempt at adding lights.","user":{"login":"simse","id":6843486,"avatar_url":"https://avatars1.githubusercontent.com/u/6843486?v=4","gravatar_id":"","url":"https://api.github.com/users/simse","html_url":"https://github.com/simse","followers_url":"https://api.github.com/users/simse/followers","following_url":"https://api.github.com/users/simse/following{/other_user}","gists_url":"https://api.github.com/users/simse/gists{/gist_id}","starred_url":"https://api.github.com/users/simse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simse/subscriptions","organizations_url":"https://api.github.com/users/simse/orgs","repos_url":"https://api.github.com/users/simse/repos","events_url":"https://api.github.com/users/simse/events{/privacy}","received_events_url":"https://api.github.com/users/simse/received_events","type":"User","site_admin":false},"body":"## Description:\r\nStarted adding lights. *However, I need help* The light isn't showing up in the home app. I am aware that it wouldn't work, but I would assume that it would at least show up. What am I missing here? Also, sorry for the messy code, I was trying to debug.\r\n\r\n## Checklist:\r\n  - [ ] The code change is tested and works locally.\r\n\r\nIf user exposed functionality or configuration variables are added/changed:\r\n  - [ ] Documentation added/updated in [home-assistant.github.io](https://github.com/home-assistant/home-assistant.github.io)\r\n\r\nIf the code communicates with devices, web services, or third-party tools:\r\n  - [ ] Local tests with `tox` run successfully. **Your PR cannot be merged unless tests pass**\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 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  - [ ] Local tests with `tox` run successfully. **Your PR cannot be merged unless tests pass**\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","created_at":"2018-02-21T13:58:47Z","updated_at":"2018-02-21T13:58:56Z","closed_at":null,"merged_at":null,"merge_commit_sha":"ea490e4b1f15e91f33fbd4daab9c31e007295e6e","assignee":null,"assignees":[],"requested_reviewers":[{"login":"cdce8p","id":30130371,"avatar_url":"https://avatars2.githubusercontent.com/u/30130371?v=4","gravatar_id":"","url":"https://api.github.com/users/cdce8p","html_url":"https://github.com/cdce8p","followers_url":"https://api.github.com/users/cdce8p/followers","following_url":"https://api.github.com/users/cdce8p/following{/other_user}","gists_url":"https://api.github.com/users/cdce8p/gists{/gist_id}","starred_url":"https://api.github.com/users/cdce8p/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cdce8p/subscriptions","organizations_url":"https://api.github.com/users/cdce8p/orgs","repos_url":"https://api.github.com/users/cdce8p/repos","events_url":"https://api.github.com/users/cdce8p/events{/privacy}","received_events_url":"https://api.github.com/users/cdce8p/received_events","type":"User","site_admin":false}],"requested_teams":[],"labels":[{"id":527909495,"url":"https://api.github.com/repos/home-assistant/home-assistant/labels/cla-needed","name":"cla-needed","color":"ffff00","default":false},{"id":840363345,"url":"https://api.github.com/repos/home-assistant/home-assistant/labels/component:%20homekit","name":"component: homekit","color":"ededed","default":false},{"id":526146744,"url":"https://api.github.com/repos/home-assistant/home-assistant/labels/new-platform","name":"new-platform","color":"ededed","default":false},{"id":840363347,"url":"https://api.github.com/repos/home-assistant/home-assistant/labels/platform:%20homekit.const","name":"platform: homekit.const","color":"ededed","default":false},{"id":843897245,"url":"https://api.github.com/repos/home-assistant/home-assistant/labels/platform:%20homekit.lights","name":"platform: homekit.lights","color":"ededed","default":false}],"milestone":null,"commits_url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576/commits","review_comments_url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576/comments","review_comment_url":"https://api.github.com/repos/home-assistant/home-assistant/pulls/comments{/number}","comments_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/12576/comments","statuses_url":"https://api.github.com/repos/home-assistant/home-assistant/statuses/e9dfe0915b1e3d8b286a05203a52f0ed8640dffc","head":{"label":"simse:dev","ref":"dev","sha":"e9dfe0915b1e3d8b286a05203a52f0ed8640dffc","user":{"login":"simse","id":6843486,"avatar_url":"https://avatars1.githubusercontent.com/u/6843486?v=4","gravatar_id":"","url":"https://api.github.com/users/simse","html_url":"https://github.com/simse","followers_url":"https://api.github.com/users/simse/followers","following_url":"https://api.github.com/users/simse/following{/other_user}","gists_url":"https://api.github.com/users/simse/gists{/gist_id}","starred_url":"https://api.github.com/users/simse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simse/subscriptions","organizations_url":"https://api.github.com/users/simse/orgs","repos_url":"https://api.github.com/users/simse/repos","events_url":"https://api.github.com/users/simse/events{/privacy}","received_events_url":"https://api.github.com/users/simse/received_events","type":"User","site_admin":false},"repo":{"id":122234548,"name":"home-assistant","full_name":"simse/home-assistant","owner":{"login":"simse","id":6843486,"avatar_url":"https://avatars1.githubusercontent.com/u/6843486?v=4","gravatar_id":"","url":"https://api.github.com/users/simse","html_url":"https://github.com/simse","followers_url":"https://api.github.com/users/simse/followers","following_url":"https://api.github.com/users/simse/following{/other_user}","gists_url":"https://api.github.com/users/simse/gists{/gist_id}","starred_url":"https://api.github.com/users/simse/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/simse/subscriptions","organizations_url":"https://api.github.com/users/simse/orgs","repos_url":"https://api.github.com/users/simse/repos","events_url":"https://api.github.com/users/simse/events{/privacy}","received_events_url":"https://api.github.com/users/simse/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/simse/home-assistant","description":":house_with_garden: Open-source home automation platform running on Python 3","fork":true,"url":"https://api.github.com/repos/simse/home-assistant","forks_url":"https://api.github.com/repos/simse/home-assistant/forks","keys_url":"https://api.github.com/repos/simse/home-assistant/keys{/key_id}","collaborators_url":"https://api.github.com/repos/simse/home-assistant/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/simse/home-assistant/teams","hooks_url":"https://api.github.com/repos/simse/home-assistant/hooks","issue_events_url":"https://api.github.com/repos/simse/home-assistant/issues/events{/number}","events_url":"https://api.github.com/repos/simse/home-assistant/events","assignees_url":"https://api.github.com/repos/simse/home-assistant/assignees{/user}","branches_url":"https://api.github.com/repos/simse/home-assistant/branches{/branch}","tags_url":"https://api.github.com/repos/simse/home-assistant/tags","blobs_url":"https://api.github.com/repos/simse/home-assistant/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/simse/home-assistant/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/simse/home-assistant/git/refs{/sha}","trees_url":"https://api.github.com/repos/simse/home-assistant/git/trees{/sha}","statuses_url":"https://api.github.com/repos/simse/home-assistant/statuses/{sha}","languages_url":"https://api.github.com/repos/simse/home-assistant/languages","stargazers_url":"https://api.github.com/repos/simse/home-assistant/stargazers","contributors_url":"https://api.github.com/repos/simse/home-assistant/contributors","subscribers_url":"https://api.github.com/repos/simse/home-assistant/subscribers","subscription_url":"https://api.github.com/repos/simse/home-assistant/subscription","commits_url":"https://api.github.com/repos/simse/home-assistant/commits{/sha}","git_commits_url":"https://api.github.com/repos/simse/home-assistant/git/commits{/sha}","comments_url":"https://api.github.com/repos/simse/home-assistant/comments{/number}","issue_comment_url":"https://api.github.com/repos/simse/home-assistant/issues/comments{/number}","contents_url":"https://api.github.com/repos/simse/home-assistant/contents/{+path}","compare_url":"https://api.github.com/repos/simse/home-assistant/compare/{base}...{head}","merges_url":"https://api.github.com/repos/simse/home-assistant/merges","archive_url":"https://api.github.com/repos/simse/home-assistant/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/simse/home-assistant/downloads","issues_url":"https://api.github.com/repos/simse/home-assistant/issues{/number}","pulls_url":"https://api.github.com/repos/simse/home-assistant/pulls{/number}","milestones_url":"https://api.github.com/repos/simse/home-assistant/milestones{/number}","notifications_url":"https://api.github.com/repos/simse/home-assistant/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/simse/home-assistant/labels{/name}","releases_url":"https://api.github.com/repos/simse/home-assistant/releases{/id}","deployments_url":"https://api.github.com/repos/simse/home-assistant/deployments","created_at":"2018-02-20T17:58:46Z","updated_at":"2018-02-20T17:58:54Z","pushed_at":"2018-02-21T13:57:04Z","git_url":"git://github.com/simse/home-assistant.git","ssh_url":"git@github.com:simse/home-assistant.git","clone_url":"https://github.com/simse/home-assistant.git","svn_url":"https://github.com/simse/home-assistant","homepage":"https://home-assistant.io","size":78747,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"other","name":"Other","spdx_id":null,"url":null},"forks":0,"open_issues":0,"watchers":0,"default_branch":"dev"}},"base":{"label":"home-assistant:dev","ref":"dev","sha":"8d0b7adf24aae1954b9bff4e6a93489bf1009a39","user":{"login":"home-assistant","id":13844975,"avatar_url":"https://avatars3.githubusercontent.com/u/13844975?v=4","gravatar_id":"","url":"https://api.github.com/users/home-assistant","html_url":"https://github.com/home-assistant","followers_url":"https://api.github.com/users/home-assistant/followers","following_url":"https://api.github.com/users/home-assistant/following{/other_user}","gists_url":"https://api.github.com/users/home-assistant/gists{/gist_id}","starred_url":"https://api.github.com/users/home-assistant/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/home-assistant/subscriptions","organizations_url":"https://api.github.com/users/home-assistant/orgs","repos_url":"https://api.github.com/users/home-assistant/repos","events_url":"https://api.github.com/users/home-assistant/events{/privacy}","received_events_url":"https://api.github.com/users/home-assistant/received_events","type":"Organization","site_admin":false},"repo":{"id":12888993,"name":"home-assistant","full_name":"home-assistant/home-assistant","owner":{"login":"home-assistant","id":13844975,"avatar_url":"https://avatars3.githubusercontent.com/u/13844975?v=4","gravatar_id":"","url":"https://api.github.com/users/home-assistant","html_url":"https://github.com/home-assistant","followers_url":"https://api.github.com/users/home-assistant/followers","following_url":"https://api.github.com/users/home-assistant/following{/other_user}","gists_url":"https://api.github.com/users/home-assistant/gists{/gist_id}","starred_url":"https://api.github.com/users/home-assistant/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/home-assistant/subscriptions","organizations_url":"https://api.github.com/users/home-assistant/orgs","repos_url":"https://api.github.com/users/home-assistant/repos","events_url":"https://api.github.com/users/home-assistant/events{/privacy}","received_events_url":"https://api.github.com/users/home-assistant/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/home-assistant/home-assistant","description":":house_with_garden: Open-source home automation platform running on Python 3","fork":false,"url":"https://api.github.com/repos/home-assistant/home-assistant","forks_url":"https://api.github.com/repos/home-assistant/home-assistant/forks","keys_url":"https://api.github.com/repos/home-assistant/home-assistant/keys{/key_id}","collaborators_url":"https://api.github.com/repos/home-assistant/home-assistant/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/home-assistant/home-assistant/teams","hooks_url":"https://api.github.com/repos/home-assistant/home-assistant/hooks","issue_events_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/events{/number}","events_url":"https://api.github.com/repos/home-assistant/home-assistant/events","assignees_url":"https://api.github.com/repos/home-assistant/home-assistant/assignees{/user}","branches_url":"https://api.github.com/repos/home-assistant/home-assistant/branches{/branch}","tags_url":"https://api.github.com/repos/home-assistant/home-assistant/tags","blobs_url":"https://api.github.com/repos/home-assistant/home-assistant/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/home-assistant/home-assistant/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/home-assistant/home-assistant/git/refs{/sha}","trees_url":"https://api.github.com/repos/home-assistant/home-assistant/git/trees{/sha}","statuses_url":"https://api.github.com/repos/home-assistant/home-assistant/statuses/{sha}","languages_url":"https://api.github.com/repos/home-assistant/home-assistant/languages","stargazers_url":"https://api.github.com/repos/home-assistant/home-assistant/stargazers","contributors_url":"https://api.github.com/repos/home-assistant/home-assistant/contributors","subscribers_url":"https://api.github.com/repos/home-assistant/home-assistant/subscribers","subscription_url":"https://api.github.com/repos/home-assistant/home-assistant/subscription","commits_url":"https://api.github.com/repos/home-assistant/home-assistant/commits{/sha}","git_commits_url":"https://api.github.com/repos/home-assistant/home-assistant/git/commits{/sha}","comments_url":"https://api.github.com/repos/home-assistant/home-assistant/comments{/number}","issue_comment_url":"https://api.github.com/repos/home-assistant/home-assistant/issues/comments{/number}","contents_url":"https://api.github.com/repos/home-assistant/home-assistant/contents/{+path}","compare_url":"https://api.github.com/repos/home-assistant/home-assistant/compare/{base}...{head}","merges_url":"https://api.github.com/repos/home-assistant/home-assistant/merges","archive_url":"https://api.github.com/repos/home-assistant/home-assistant/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/home-assistant/home-assistant/downloads","issues_url":"https://api.github.com/repos/home-assistant/home-assistant/issues{/number}","pulls_url":"https://api.github.com/repos/home-assistant/home-assistant/pulls{/number}","milestones_url":"https://api.github.com/repos/home-assistant/home-assistant/milestones{/number}","notifications_url":"https://api.github.com/repos/home-assistant/home-assistant/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/home-assistant/home-assistant/labels{/name}","releases_url":"https://api.github.com/repos/home-assistant/home-assistant/releases{/id}","deployments_url":"https://api.github.com/repos/home-assistant/home-assistant/deployments","created_at":"2013-09-17T07:29:48Z","updated_at":"2018-02-21T12:51:36Z","pushed_at":"2018-02-21T13:58:48Z","git_url":"git://github.com/home-assistant/home-assistant.git","ssh_url":"git@github.com:home-assistant/home-assistant.git","clone_url":"https://github.com/home-assistant/home-assistant.git","svn_url":"https://github.com/home-assistant/home-assistant","homepage":"https://home-assistant.io","size":78797,"stargazers_count":12672,"watchers_count":12672,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":false,"has_pages":false,"forks_count":3664,"mirror_url":null,"archived":false,"open_issues_count":831,"license":{"key":"other","name":"Other","spdx_id":null,"url":null},"forks":3664,"open_issues":831,"watchers":12672,"default_branch":"dev"}},"_links":{"self":{"href":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576"},"html":{"href":"https://github.com/home-assistant/home-assistant/pull/12576"},"issue":{"href":"https://api.github.com/repos/home-assistant/home-assistant/issues/12576"},"comments":{"href":"https://api.github.com/repos/home-assistant/home-assistant/issues/12576/comments"},"review_comments":{"href":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576/comments"},"review_comment":{"href":"https://api.github.com/repos/home-assistant/home-assistant/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/home-assistant/home-assistant/pulls/12576/commits"},"statuses":{"href":"https://api.github.com/repos/home-assistant/home-assistant/statuses/e9dfe0915b1e3d8b286a05203a52f0ed8640dffc"}},"author_association":"CONTRIBUTOR"}}
{"actor":{"display_login":"houndci-bot"}}