{"action":"created","issue":{"url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/issues/6","repository_url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery","labels_url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/issues/6/labels{/name}","comments_url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/issues/6/comments","events_url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/issues/6/events","html_url":"https://github.com/muhammadzaheer/rl_laplacian_option_discovery/pull/6","id":274445685,"number":6,"title":"Modularized options","user":{"login":"muhammadzaheer","id":8004953,"avatar_url":"https://avatars0.githubusercontent.com/u/8004953?v=4","gravatar_id":"","url":"https://api.github.com/users/muhammadzaheer","html_url":"https://github.com/muhammadzaheer","followers_url":"https://api.github.com/users/muhammadzaheer/followers","following_url":"https://api.github.com/users/muhammadzaheer/following{/other_user}","gists_url":"https://api.github.com/users/muhammadzaheer/gists{/gist_id}","starred_url":"https://api.github.com/users/muhammadzaheer/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/muhammadzaheer/subscriptions","organizations_url":"https://api.github.com/users/muhammadzaheer/orgs","repos_url":"https://api.github.com/users/muhammadzaheer/repos","events_url":"https://api.github.com/users/muhammadzaheer/events{/privacy}","received_events_url":"https://api.github.com/users/muhammadzaheer/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"sungsulim","id":11016779,"avatar_url":"https://avatars0.githubusercontent.com/u/11016779?v=4","gravatar_id":"","url":"https://api.github.com/users/sungsulim","html_url":"https://github.com/sungsulim","followers_url":"https://api.github.com/users/sungsulim/followers","following_url":"https://api.github.com/users/sungsulim/following{/other_user}","gists_url":"https://api.github.com/users/sungsulim/gists{/gist_id}","starred_url":"https://api.github.com/users/sungsulim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sungsulim/subscriptions","organizations_url":"https://api.github.com/users/sungsulim/orgs","repos_url":"https://api.github.com/users/sungsulim/repos","events_url":"https://api.github.com/users/sungsulim/events{/privacy}","received_events_url":"https://api.github.com/users/sungsulim/received_events","type":"User","site_admin":false},"assignees":[{"login":"sungsulim","id":11016779,"avatar_url":"https://avatars0.githubusercontent.com/u/11016779?v=4","gravatar_id":"","url":"https://api.github.com/users/sungsulim","html_url":"https://github.com/sungsulim","followers_url":"https://api.github.com/users/sungsulim/followers","following_url":"https://api.github.com/users/sungsulim/following{/other_user}","gists_url":"https://api.github.com/users/sungsulim/gists{/gist_id}","starred_url":"https://api.github.com/users/sungsulim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sungsulim/subscriptions","organizations_url":"https://api.github.com/users/sungsulim/orgs","repos_url":"https://api.github.com/users/sungsulim/repos","events_url":"https://api.github.com/users/sungsulim/events{/privacy}","received_events_url":"https://api.github.com/users/sungsulim/received_events","type":"User","site_admin":false}],"milestone":null,"comments":1,"created_at":"2017-11-16T09:22:32Z","updated_at":"2017-11-17T01:14:51Z","closed_at":null,"author_association":"OWNER","pull_request":{"url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/pulls/6","html_url":"https://github.com/muhammadzaheer/rl_laplacian_option_discovery/pull/6","diff_url":"https://github.com/muhammadzaheer/rl_laplacian_option_discovery/pull/6.diff","patch_url":"https://github.com/muhammadzaheer/rl_laplacian_option_discovery/pull/6.patch"},"body":"1. Modularized options by adding the module options.py\r\n2. Moved RLGlue, QAgent and Environment classes in the options module\r\n3. Made sure options module conforms to pep8"},"comment":{"url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/issues/comments/345115261","html_url":"https://github.com/muhammadzaheer/rl_laplacian_option_discovery/pull/6#issuecomment-345115261","issue_url":"https://api.github.com/repos/muhammadzaheer/rl_laplacian_option_discovery/issues/6","id":345115261,"user":{"login":"sungsulim","id":11016779,"avatar_url":"https://avatars0.githubusercontent.com/u/11016779?v=4","gravatar_id":"","url":"https://api.github.com/users/sungsulim","html_url":"https://github.com/sungsulim","followers_url":"https://api.github.com/users/sungsulim/followers","following_url":"https://api.github.com/users/sungsulim/following{/other_user}","gists_url":"https://api.github.com/users/sungsulim/gists{/gist_id}","starred_url":"https://api.github.com/users/sungsulim/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sungsulim/subscriptions","organizations_url":"https://api.github.com/users/sungsulim/orgs","repos_url":"https://api.github.com/users/sungsulim/repos","events_url":"https://api.github.com/users/sungsulim/events{/privacy}","received_events_url":"https://api.github.com/users/sungsulim/received_events","type":"User","site_admin":false},"created_at":"2017-11-17T01:14:51Z","updated_at":"2017-11-17T01:14:51Z","author_association":"COLLABORATOR","body":"I agree that we need a separate internal environment for learning the eigen-behavior. But I feel \r\n that instead of redundantly defining the same environment, it would be better to inherit the external environment and learn the eigen-behavior. The internal environment would have modified reward function and terminal condition. Later we would also need to switch different environments easily."}}
{"actor":{"display_login":"sungsulim"}}