{"action":"created","comment":{"url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/comments/83528249","pull_request_review_id":4369076,"id":83528249,"diff_hunk":"@@ -1,39 +1,43 @@\n-\"\"\"\n-This is the imSim program, used to drive GalSim to simulate the LSST.  Written\n-for the DESC collaboration and LSST project.  This version of the program can\n-read phoSim instance files as is. It leverages the LSST Sims GalSim interface\n-code found in sims_GalSimInterface.\n-\"\"\"\n-import os\n import sys\n-import logging\n-import argparse\n+from collections import namedtuple","path":"python/desc/imsim/imSim.py","position":11,"original_position":11,"commit_id":"39cf5869d01afd6c0e5339662ab3b6eee4f8e4e3","original_commit_id":"39cf5869d01afd6c0e5339662ab3b6eee4f8e4e3","user":{"login":"cwwalter","id":5562740,"avatar_url":"https://avatars.githubusercontent.com/u/5562740?v=3","gravatar_id":"","url":"https://api.github.com/users/cwwalter","html_url":"https://github.com/cwwalter","followers_url":"https://api.github.com/users/cwwalter/followers","following_url":"https://api.github.com/users/cwwalter/following{/other_user}","gists_url":"https://api.github.com/users/cwwalter/gists{/gist_id}","starred_url":"https://api.github.com/users/cwwalter/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cwwalter/subscriptions","organizations_url":"https://api.github.com/users/cwwalter/orgs","repos_url":"https://api.github.com/users/cwwalter/repos","events_url":"https://api.github.com/users/cwwalter/events{/privacy}","received_events_url":"https://api.github.com/users/cwwalter/received_events","type":"User","site_admin":false},"body":"Since this file now just contains the parser should it have Parser type name (instead if imSim.py) instead?  Or is the intention to put a lot of code in here?\r\n","created_at":"2016-10-15T08:22:35Z","updated_at":"2016-10-15T08:22:35Z","html_url":"https://github.com/DarkEnergyScienceCollaboration/imSim/pull/1#discussion_r83528249","pull_request_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1","_links":{"self":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/comments/83528249"},"html":{"href":"https://github.com/DarkEnergyScienceCollaboration/imSim/pull/1#discussion_r83528249"},"pull_request":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1"}}},"pull_request":{"url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1","id":89477434,"html_url":"https://github.com/DarkEnergyScienceCollaboration/imSim/pull/1","diff_url":"https://github.com/DarkEnergyScienceCollaboration/imSim/pull/1.diff","patch_url":"https://github.com/DarkEnergyScienceCollaboration/imSim/pull/1.patch","issue_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/1","number":1,"state":"open","locked":false,"title":"refactor and eupsify repo","user":{"login":"jchiang87","id":1994473,"avatar_url":"https://avatars.githubusercontent.com/u/1994473?v=3","gravatar_id":"","url":"https://api.github.com/users/jchiang87","html_url":"https://github.com/jchiang87","followers_url":"https://api.github.com/users/jchiang87/followers","following_url":"https://api.github.com/users/jchiang87/following{/other_user}","gists_url":"https://api.github.com/users/jchiang87/gists{/gist_id}","starred_url":"https://api.github.com/users/jchiang87/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jchiang87/subscriptions","organizations_url":"https://api.github.com/users/jchiang87/orgs","repos_url":"https://api.github.com/users/jchiang87/repos","events_url":"https://api.github.com/users/jchiang87/events{/privacy}","received_events_url":"https://api.github.com/users/jchiang87/received_events","type":"User","site_admin":false},"body":"These changes enable this package to be setup using eups.   I've refactored the parsing code, adding some validation of the input commands, and added unit tests.","created_at":"2016-10-15T01:43:12Z","updated_at":"2016-10-15T08:22:35Z","closed_at":null,"merged_at":null,"merge_commit_sha":"071650fa73db040d5b279897dc63cd5b50f5bca4","assignee":null,"assignees":[],"milestone":null,"commits_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1/commits","review_comments_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1/comments","review_comment_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/comments{/number}","comments_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/1/comments","statuses_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/statuses/39cf5869d01afd6c0e5339662ab3b6eee4f8e4e3","head":{"label":"DarkEnergyScienceCollaboration:u/jchiang/refactor_and_eupsify_repo","ref":"u/jchiang/refactor_and_eupsify_repo","sha":"39cf5869d01afd6c0e5339662ab3b6eee4f8e4e3","user":{"login":"DarkEnergyScienceCollaboration","id":2731443,"avatar_url":"https://avatars.githubusercontent.com/u/2731443?v=3","gravatar_id":"","url":"https://api.github.com/users/DarkEnergyScienceCollaboration","html_url":"https://github.com/DarkEnergyScienceCollaboration","followers_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/followers","following_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/following{/other_user}","gists_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/gists{/gist_id}","starred_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/subscriptions","organizations_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/orgs","repos_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/repos","events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/events{/privacy}","received_events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/received_events","type":"Organization","site_admin":false},"repo":{"id":70957320,"name":"imSim","full_name":"DarkEnergyScienceCollaboration/imSim","owner":{"login":"DarkEnergyScienceCollaboration","id":2731443,"avatar_url":"https://avatars.githubusercontent.com/u/2731443?v=3","gravatar_id":"","url":"https://api.github.com/users/DarkEnergyScienceCollaboration","html_url":"https://github.com/DarkEnergyScienceCollaboration","followers_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/followers","following_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/following{/other_user}","gists_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/gists{/gist_id}","starred_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/subscriptions","organizations_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/orgs","repos_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/repos","events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/events{/privacy}","received_events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/DarkEnergyScienceCollaboration/imSim","description":"GalSim based LSST simulation package","fork":false,"url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim","forks_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/forks","keys_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/keys{/key_id}","collaborators_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/teams","hooks_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/hooks","issue_events_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/events{/number}","events_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/events","assignees_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/assignees{/user}","branches_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/branches{/branch}","tags_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/tags","blobs_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/refs{/sha}","trees_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/trees{/sha}","statuses_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/statuses/{sha}","languages_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/languages","stargazers_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/stargazers","contributors_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/contributors","subscribers_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/subscribers","subscription_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/subscription","commits_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/commits{/sha}","git_commits_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/commits{/sha}","comments_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/comments{/number}","issue_comment_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/comments{/number}","contents_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/contents/{+path}","compare_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/compare/{base}...{head}","merges_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/merges","archive_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/downloads","issues_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues{/number}","pulls_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls{/number}","milestones_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/milestones{/number}","notifications_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/labels{/name}","releases_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/releases{/id}","deployments_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/deployments","created_at":"2016-10-15T00:53:14Z","updated_at":"2016-10-15T00:55:07Z","pushed_at":"2016-10-15T01:43:12Z","git_url":"git://github.com/DarkEnergyScienceCollaboration/imSim.git","ssh_url":"git@github.com:DarkEnergyScienceCollaboration/imSim.git","clone_url":"https://github.com/DarkEnergyScienceCollaboration/imSim.git","svn_url":"https://github.com/DarkEnergyScienceCollaboration/imSim","homepage":null,"size":18,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"base":{"label":"DarkEnergyScienceCollaboration:master","ref":"master","sha":"3d8c55f73982db91c88c2ddcbaf3716291d567f2","user":{"login":"DarkEnergyScienceCollaboration","id":2731443,"avatar_url":"https://avatars.githubusercontent.com/u/2731443?v=3","gravatar_id":"","url":"https://api.github.com/users/DarkEnergyScienceCollaboration","html_url":"https://github.com/DarkEnergyScienceCollaboration","followers_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/followers","following_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/following{/other_user}","gists_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/gists{/gist_id}","starred_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/subscriptions","organizations_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/orgs","repos_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/repos","events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/events{/privacy}","received_events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/received_events","type":"Organization","site_admin":false},"repo":{"id":70957320,"name":"imSim","full_name":"DarkEnergyScienceCollaboration/imSim","owner":{"login":"DarkEnergyScienceCollaboration","id":2731443,"avatar_url":"https://avatars.githubusercontent.com/u/2731443?v=3","gravatar_id":"","url":"https://api.github.com/users/DarkEnergyScienceCollaboration","html_url":"https://github.com/DarkEnergyScienceCollaboration","followers_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/followers","following_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/following{/other_user}","gists_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/gists{/gist_id}","starred_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/subscriptions","organizations_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/orgs","repos_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/repos","events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/events{/privacy}","received_events_url":"https://api.github.com/users/DarkEnergyScienceCollaboration/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/DarkEnergyScienceCollaboration/imSim","description":"GalSim based LSST simulation package","fork":false,"url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim","forks_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/forks","keys_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/keys{/key_id}","collaborators_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/teams","hooks_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/hooks","issue_events_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/events{/number}","events_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/events","assignees_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/assignees{/user}","branches_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/branches{/branch}","tags_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/tags","blobs_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/refs{/sha}","trees_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/trees{/sha}","statuses_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/statuses/{sha}","languages_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/languages","stargazers_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/stargazers","contributors_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/contributors","subscribers_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/subscribers","subscription_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/subscription","commits_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/commits{/sha}","git_commits_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/git/commits{/sha}","comments_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/comments{/number}","issue_comment_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/comments{/number}","contents_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/contents/{+path}","compare_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/compare/{base}...{head}","merges_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/merges","archive_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/downloads","issues_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues{/number}","pulls_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls{/number}","milestones_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/milestones{/number}","notifications_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/labels{/name}","releases_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/releases{/id}","deployments_url":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/deployments","created_at":"2016-10-15T00:53:14Z","updated_at":"2016-10-15T00:55:07Z","pushed_at":"2016-10-15T01:43:12Z","git_url":"git://github.com/DarkEnergyScienceCollaboration/imSim.git","ssh_url":"git@github.com:DarkEnergyScienceCollaboration/imSim.git","clone_url":"https://github.com/DarkEnergyScienceCollaboration/imSim.git","svn_url":"https://github.com/DarkEnergyScienceCollaboration/imSim","homepage":null,"size":18,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":1,"forks":0,"open_issues":1,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1"},"html":{"href":"https://github.com/DarkEnergyScienceCollaboration/imSim/pull/1"},"issue":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/1"},"comments":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/issues/1/comments"},"review_comments":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1/comments"},"review_comment":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/pulls/1/commits"},"statuses":{"href":"https://api.github.com/repos/DarkEnergyScienceCollaboration/imSim/statuses/39cf5869d01afd6c0e5339662ab3b6eee4f8e4e3"}}}}
{"actor":{"display_login":"cwwalter"}}