{"action":"created","comment":{"url":"https://api.github.com/repos/Abjad/abjad/pulls/comments/74357289","id":74357289,"diff_hunk":"@@ -172,7 +174,8 @@ def __format__(self, format_specification=''):\n             return self._lilypond_format\n         elif format_specification == 'storage':\n             return systemtools.StorageFormatManager.get_storage_format(self)\n-        return str(self)\n+        else:","path":"abjad/tools/schemetools/Scheme.py","position":47,"original_position":47,"commit_id":"a810d7900c46d16a829bda862381338c98c3d9b3","original_commit_id":"a810d7900c46d16a829bda862381338c98c3d9b3","user":{"login":"josiah-wolf-oberholtzer","id":5505674,"avatar_url":"https://avatars.githubusercontent.com/u/5505674?v=3","gravatar_id":"","url":"https://api.github.com/users/josiah-wolf-oberholtzer","html_url":"https://github.com/josiah-wolf-oberholtzer","followers_url":"https://api.github.com/users/josiah-wolf-oberholtzer/followers","following_url":"https://api.github.com/users/josiah-wolf-oberholtzer/following{/other_user}","gists_url":"https://api.github.com/users/josiah-wolf-oberholtzer/gists{/gist_id}","starred_url":"https://api.github.com/users/josiah-wolf-oberholtzer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/josiah-wolf-oberholtzer/subscriptions","organizations_url":"https://api.github.com/users/josiah-wolf-oberholtzer/orgs","repos_url":"https://api.github.com/users/josiah-wolf-oberholtzer/repos","events_url":"https://api.github.com/users/josiah-wolf-oberholtzer/events{/privacy}","received_events_url":"https://api.github.com/users/josiah-wolf-oberholtzer/received_events","type":"User","site_admin":false},"body":"You don't need the `else:`. It will fall through.","created_at":"2016-08-11T00:49:41Z","updated_at":"2016-08-11T00:49:41Z","html_url":"https://github.com/Abjad/abjad/pull/744#discussion_r74357289","pull_request_url":"https://api.github.com/repos/Abjad/abjad/pulls/744","_links":{"self":{"href":"https://api.github.com/repos/Abjad/abjad/pulls/comments/74357289"},"html":{"href":"https://github.com/Abjad/abjad/pull/744#discussion_r74357289"},"pull_request":{"href":"https://api.github.com/repos/Abjad/abjad/pulls/744"}}},"pull_request":{"url":"https://api.github.com/repos/Abjad/abjad/pulls/744","id":80798104,"html_url":"https://github.com/Abjad/abjad/pull/744","diff_url":"https://github.com/Abjad/abjad/pull/744.diff","patch_url":"https://github.com/Abjad/abjad/pull/744.patch","issue_url":"https://api.github.com/repos/Abjad/abjad/issues/744","number":744,"state":"open","locked":false,"title":"Minor cleanups in schemetools.Scheme","user":{"login":"ajyoon","id":15976214,"avatar_url":"https://avatars.githubusercontent.com/u/15976214?v=3","gravatar_id":"","url":"https://api.github.com/users/ajyoon","html_url":"https://github.com/ajyoon","followers_url":"https://api.github.com/users/ajyoon/followers","following_url":"https://api.github.com/users/ajyoon/following{/other_user}","gists_url":"https://api.github.com/users/ajyoon/gists{/gist_id}","starred_url":"https://api.github.com/users/ajyoon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajyoon/subscriptions","organizations_url":"https://api.github.com/users/ajyoon/orgs","repos_url":"https://api.github.com/users/ajyoon/repos","events_url":"https://api.github.com/users/ajyoon/events{/privacy}","received_events_url":"https://api.github.com/users/ajyoon/received_events","type":"User","site_admin":false},"body":"Addresses #405 and various minor cleanups in the vicinity - I understand this was assigned but it smelled stale, feel free to close if this has been resolved in another branch.\r\n\r\nOn a related note, I noticed several of the methods in the `Scheme` class import the `Scheme` class itself and explicitly refer to `schemetools.Scheme` - isn't this redundant? Why not refer to `Scheme` directly, or even `cls`? Or is there some Python import voodoo being accounted for that I'm not aware of?\r\n\r\nFor example:\r\n\r\n```python\r\n    @staticmethod\r\n    def format_embedded_scheme_value(value, force_quotes=False):\r\n        r'''Formats `value` as an embedded Scheme value.\r\n        '''\r\n        from abjad.tools import datastructuretools\r\n        from abjad.tools import schemetools\r\n        if isinstance(value, datastructuretools.OrdinalConstant):\r\n            result = '#' + repr(value).lower()\r\n        else:\r\n            result = schemetools.Scheme.format_scheme_value(\r\n                value, force_quotes=force_quotes)\r\n            if ((isinstance(value, bool)) or\r\n                    (isinstance(value, str) and not force_quotes) or\r\n                    (isinstance(value, schemetools.Scheme))):\r\n                result = '#' + result\r\n        return result\r\n```","created_at":"2016-08-10T13:54:24Z","updated_at":"2016-08-11T00:49:41Z","closed_at":null,"merged_at":null,"merge_commit_sha":"46f195d9aca0ab6f6dd5779bd4ab0e9ecf933771","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/Abjad/abjad/pulls/744/commits","review_comments_url":"https://api.github.com/repos/Abjad/abjad/pulls/744/comments","review_comment_url":"https://api.github.com/repos/Abjad/abjad/pulls/comments{/number}","comments_url":"https://api.github.com/repos/Abjad/abjad/issues/744/comments","statuses_url":"https://api.github.com/repos/Abjad/abjad/statuses/a810d7900c46d16a829bda862381338c98c3d9b3","head":{"label":"ajyoon:scheme-init-cleanup","ref":"scheme-init-cleanup","sha":"a810d7900c46d16a829bda862381338c98c3d9b3","user":{"login":"ajyoon","id":15976214,"avatar_url":"https://avatars.githubusercontent.com/u/15976214?v=3","gravatar_id":"","url":"https://api.github.com/users/ajyoon","html_url":"https://github.com/ajyoon","followers_url":"https://api.github.com/users/ajyoon/followers","following_url":"https://api.github.com/users/ajyoon/following{/other_user}","gists_url":"https://api.github.com/users/ajyoon/gists{/gist_id}","starred_url":"https://api.github.com/users/ajyoon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajyoon/subscriptions","organizations_url":"https://api.github.com/users/ajyoon/orgs","repos_url":"https://api.github.com/users/ajyoon/repos","events_url":"https://api.github.com/users/ajyoon/events{/privacy}","received_events_url":"https://api.github.com/users/ajyoon/received_events","type":"User","site_admin":false},"repo":{"id":65380085,"name":"abjad","full_name":"ajyoon/abjad","owner":{"login":"ajyoon","id":15976214,"avatar_url":"https://avatars.githubusercontent.com/u/15976214?v=3","gravatar_id":"","url":"https://api.github.com/users/ajyoon","html_url":"https://github.com/ajyoon","followers_url":"https://api.github.com/users/ajyoon/followers","following_url":"https://api.github.com/users/ajyoon/following{/other_user}","gists_url":"https://api.github.com/users/ajyoon/gists{/gist_id}","starred_url":"https://api.github.com/users/ajyoon/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ajyoon/subscriptions","organizations_url":"https://api.github.com/users/ajyoon/orgs","repos_url":"https://api.github.com/users/ajyoon/repos","events_url":"https://api.github.com/users/ajyoon/events{/privacy}","received_events_url":"https://api.github.com/users/ajyoon/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/ajyoon/abjad","description":"Abjad is a Python API for Formalized Score Control","fork":true,"url":"https://api.github.com/repos/ajyoon/abjad","forks_url":"https://api.github.com/repos/ajyoon/abjad/forks","keys_url":"https://api.github.com/repos/ajyoon/abjad/keys{/key_id}","collaborators_url":"https://api.github.com/repos/ajyoon/abjad/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/ajyoon/abjad/teams","hooks_url":"https://api.github.com/repos/ajyoon/abjad/hooks","issue_events_url":"https://api.github.com/repos/ajyoon/abjad/issues/events{/number}","events_url":"https://api.github.com/repos/ajyoon/abjad/events","assignees_url":"https://api.github.com/repos/ajyoon/abjad/assignees{/user}","branches_url":"https://api.github.com/repos/ajyoon/abjad/branches{/branch}","tags_url":"https://api.github.com/repos/ajyoon/abjad/tags","blobs_url":"https://api.github.com/repos/ajyoon/abjad/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/ajyoon/abjad/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/ajyoon/abjad/git/refs{/sha}","trees_url":"https://api.github.com/repos/ajyoon/abjad/git/trees{/sha}","statuses_url":"https://api.github.com/repos/ajyoon/abjad/statuses/{sha}","languages_url":"https://api.github.com/repos/ajyoon/abjad/languages","stargazers_url":"https://api.github.com/repos/ajyoon/abjad/stargazers","contributors_url":"https://api.github.com/repos/ajyoon/abjad/contributors","subscribers_url":"https://api.github.com/repos/ajyoon/abjad/subscribers","subscription_url":"https://api.github.com/repos/ajyoon/abjad/subscription","commits_url":"https://api.github.com/repos/ajyoon/abjad/commits{/sha}","git_commits_url":"https://api.github.com/repos/ajyoon/abjad/git/commits{/sha}","comments_url":"https://api.github.com/repos/ajyoon/abjad/comments{/number}","issue_comment_url":"https://api.github.com/repos/ajyoon/abjad/issues/comments{/number}","contents_url":"https://api.github.com/repos/ajyoon/abjad/contents/{+path}","compare_url":"https://api.github.com/repos/ajyoon/abjad/compare/{base}...{head}","merges_url":"https://api.github.com/repos/ajyoon/abjad/merges","archive_url":"https://api.github.com/repos/ajyoon/abjad/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/ajyoon/abjad/downloads","issues_url":"https://api.github.com/repos/ajyoon/abjad/issues{/number}","pulls_url":"https://api.github.com/repos/ajyoon/abjad/pulls{/number}","milestones_url":"https://api.github.com/repos/ajyoon/abjad/milestones{/number}","notifications_url":"https://api.github.com/repos/ajyoon/abjad/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/ajyoon/abjad/labels{/name}","releases_url":"https://api.github.com/repos/ajyoon/abjad/releases{/id}","deployments_url":"https://api.github.com/repos/ajyoon/abjad/deployments","created_at":"2016-08-10T12:19:01Z","updated_at":"2016-08-10T12:19:08Z","pushed_at":"2016-08-11T00:46:27Z","git_url":"git://github.com/ajyoon/abjad.git","ssh_url":"git@github.com:ajyoon/abjad.git","clone_url":"https://github.com/ajyoon/abjad.git","svn_url":"https://github.com/ajyoon/abjad","homepage":"http://projectabjad.org","size":343597,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"Abjad:master","ref":"master","sha":"7b65fbb6cb6fb85d5daa847611858f6ec0e05cde","user":{"login":"Abjad","id":5782401,"avatar_url":"https://avatars.githubusercontent.com/u/5782401?v=3","gravatar_id":"","url":"https://api.github.com/users/Abjad","html_url":"https://github.com/Abjad","followers_url":"https://api.github.com/users/Abjad/followers","following_url":"https://api.github.com/users/Abjad/following{/other_user}","gists_url":"https://api.github.com/users/Abjad/gists{/gist_id}","starred_url":"https://api.github.com/users/Abjad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Abjad/subscriptions","organizations_url":"https://api.github.com/users/Abjad/orgs","repos_url":"https://api.github.com/users/Abjad/repos","events_url":"https://api.github.com/users/Abjad/events{/privacy}","received_events_url":"https://api.github.com/users/Abjad/received_events","type":"Organization","site_admin":false},"repo":{"id":13958501,"name":"abjad","full_name":"Abjad/abjad","owner":{"login":"Abjad","id":5782401,"avatar_url":"https://avatars.githubusercontent.com/u/5782401?v=3","gravatar_id":"","url":"https://api.github.com/users/Abjad","html_url":"https://github.com/Abjad","followers_url":"https://api.github.com/users/Abjad/followers","following_url":"https://api.github.com/users/Abjad/following{/other_user}","gists_url":"https://api.github.com/users/Abjad/gists{/gist_id}","starred_url":"https://api.github.com/users/Abjad/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Abjad/subscriptions","organizations_url":"https://api.github.com/users/Abjad/orgs","repos_url":"https://api.github.com/users/Abjad/repos","events_url":"https://api.github.com/users/Abjad/events{/privacy}","received_events_url":"https://api.github.com/users/Abjad/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/Abjad/abjad","description":"Abjad is a Python API for Formalized Score Control","fork":false,"url":"https://api.github.com/repos/Abjad/abjad","forks_url":"https://api.github.com/repos/Abjad/abjad/forks","keys_url":"https://api.github.com/repos/Abjad/abjad/keys{/key_id}","collaborators_url":"https://api.github.com/repos/Abjad/abjad/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/Abjad/abjad/teams","hooks_url":"https://api.github.com/repos/Abjad/abjad/hooks","issue_events_url":"https://api.github.com/repos/Abjad/abjad/issues/events{/number}","events_url":"https://api.github.com/repos/Abjad/abjad/events","assignees_url":"https://api.github.com/repos/Abjad/abjad/assignees{/user}","branches_url":"https://api.github.com/repos/Abjad/abjad/branches{/branch}","tags_url":"https://api.github.com/repos/Abjad/abjad/tags","blobs_url":"https://api.github.com/repos/Abjad/abjad/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/Abjad/abjad/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/Abjad/abjad/git/refs{/sha}","trees_url":"https://api.github.com/repos/Abjad/abjad/git/trees{/sha}","statuses_url":"https://api.github.com/repos/Abjad/abjad/statuses/{sha}","languages_url":"https://api.github.com/repos/Abjad/abjad/languages","stargazers_url":"https://api.github.com/repos/Abjad/abjad/stargazers","contributors_url":"https://api.github.com/repos/Abjad/abjad/contributors","subscribers_url":"https://api.github.com/repos/Abjad/abjad/subscribers","subscription_url":"https://api.github.com/repos/Abjad/abjad/subscription","commits_url":"https://api.github.com/repos/Abjad/abjad/commits{/sha}","git_commits_url":"https://api.github.com/repos/Abjad/abjad/git/commits{/sha}","comments_url":"https://api.github.com/repos/Abjad/abjad/comments{/number}","issue_comment_url":"https://api.github.com/repos/Abjad/abjad/issues/comments{/number}","contents_url":"https://api.github.com/repos/Abjad/abjad/contents/{+path}","compare_url":"https://api.github.com/repos/Abjad/abjad/compare/{base}...{head}","merges_url":"https://api.github.com/repos/Abjad/abjad/merges","archive_url":"https://api.github.com/repos/Abjad/abjad/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/Abjad/abjad/downloads","issues_url":"https://api.github.com/repos/Abjad/abjad/issues{/number}","pulls_url":"https://api.github.com/repos/Abjad/abjad/pulls{/number}","milestones_url":"https://api.github.com/repos/Abjad/abjad/milestones{/number}","notifications_url":"https://api.github.com/repos/Abjad/abjad/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/Abjad/abjad/labels{/name}","releases_url":"https://api.github.com/repos/Abjad/abjad/releases{/id}","deployments_url":"https://api.github.com/repos/Abjad/abjad/deployments","created_at":"2013-10-29T14:43:16Z","updated_at":"2016-08-06T10:04:27Z","pushed_at":"2016-08-11T00:46:28Z","git_url":"git://github.com/Abjad/abjad.git","ssh_url":"git@github.com:Abjad/abjad.git","clone_url":"https://github.com/Abjad/abjad.git","svn_url":"https://github.com/Abjad/abjad","homepage":"http://projectabjad.org","size":343627,"stargazers_count":61,"watchers_count":61,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":18,"mirror_url":null,"open_issues_count":175,"forks":18,"open_issues":175,"watchers":61,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/Abjad/abjad/pulls/744"},"html":{"href":"https://github.com/Abjad/abjad/pull/744"},"issue":{"href":"https://api.github.com/repos/Abjad/abjad/issues/744"},"comments":{"href":"https://api.github.com/repos/Abjad/abjad/issues/744/comments"},"review_comments":{"href":"https://api.github.com/repos/Abjad/abjad/pulls/744/comments"},"review_comment":{"href":"https://api.github.com/repos/Abjad/abjad/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/Abjad/abjad/pulls/744/commits"},"statuses":{"href":"https://api.github.com/repos/Abjad/abjad/statuses/a810d7900c46d16a829bda862381338c98c3d9b3"}}}}
{"actor":{"display_login":"josiah-wolf-oberholtzer"}}