{"action":"created","comment":{"url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/comments/140606366","pull_request_review_id":64710394,"id":140606366,"diff_hunk":"@@ -422,42 +494,54 @@ def doProcessCcd(repo, calib_repo, processed_repo, visit, ccdnum):\n         The output repository location on disk where ingested calibration images live.\n     processed_repo: `str`\n         The output repository location on disk where processed raw images live.\n-    visit: `str`\n-        One or more DECam visit numbers, e.g., `54321`. Multiple visits may be\n-        indicated with standard Butler parsing, e.g., `54321^12345`.\n-    ccdnum: `str`\n-        One or more DECam CCDs (`1` through `62` are allowed). Setting\n-        ccdnum='1..62' will process all of the CCDs.\n+    dataId: `str`\n+        Butler identifier naming the data to be processed (e.g., visit and ccdnum)\n+        formatted in the usual way (e.g., 'visit=54321 ccdnum=7').\n \n     Returns\n     -------\n     process_metadata: `PropertySet` or None\n         Metadata from the ProcessCcdTask for use by ap_verify\n \n     BASH EQUIVALENT:\n-    $ processCcd.py repo --id visit=visit ccdnum=ccdnum\n+    $ processCcd.py repo --id dataId\n             --output processed_repo --calib calib_repo\n             -C $OBS_DECAM_DIR/config/processCcdCpIsr.py\n             -C processccd_config.py\n             --config calibrate.doAstrometry=True calibrate.doPhotoCal=True\n     ** to run from bash, 'processccd_config.py' must exist and contain\n        all of the refObjLoader information in the code below. repo must also\n-       already contain the ref_cats (this is done during doIngest).\n+       already contain the refcats (this is done during doIngest).\n \n     RESULT:\n     processed_repo/visit populated with subdirectories containing the\n     usual post-ISR data (bkgd, calexp, icExp, icSrc, postISR).\n     By default, the configuration for astrometric reference catalogs uses Gaia\n     and the configuration for photometry reference catalogs uses Pan-STARRS.\n     '''\n+    lsst.log.configure()\n+    log = lsst.log.Log.getLogger('ap.pipe.doProcessCcd')\n+    dataId_items = re.split('[ +=]', dataId)\n+    dataId_dict = dict(zip(dataId_items[::2], dataId_items[1::2]))","path":"python/lsst/ap/pipe/ap_pipe.py","position":527,"original_position":527,"commit_id":"a3db926292cbc39123f2332ca2e51d9be4e2b0d9","original_commit_id":"a3db926292cbc39123f2332ca2e51d9be4e2b0d9","user":{"login":"mrawls","id":5659077,"avatar_url":"https://avatars2.githubusercontent.com/u/5659077?v=4","gravatar_id":"","url":"https://api.github.com/users/mrawls","html_url":"https://github.com/mrawls","followers_url":"https://api.github.com/users/mrawls/followers","following_url":"https://api.github.com/users/mrawls/following{/other_user}","gists_url":"https://api.github.com/users/mrawls/gists{/gist_id}","starred_url":"https://api.github.com/users/mrawls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrawls/subscriptions","organizations_url":"https://api.github.com/users/mrawls/orgs","repos_url":"https://api.github.com/users/mrawls/repos","events_url":"https://api.github.com/users/mrawls/events{/privacy}","received_events_url":"https://api.github.com/users/mrawls/received_events","type":"User","site_admin":false},"body":"Yep. If `dataId` has an odd number of elements, it will just omit the last one and carry on. This seems as reasonable as anything.","created_at":"2017-09-22T22:05:37Z","updated_at":"2017-09-22T22:05:37Z","html_url":"https://github.com/lsst-dm/ap_pipe/pull/6#discussion_r140606366","pull_request_url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6","author_association":"COLLABORATOR","_links":{"self":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/comments/140606366"},"html":{"href":"https://github.com/lsst-dm/ap_pipe/pull/6#discussion_r140606366"},"pull_request":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6"}},"in_reply_to_id":140568000},"pull_request":{"url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6","id":142243817,"html_url":"https://github.com/lsst-dm/ap_pipe/pull/6","diff_url":"https://github.com/lsst-dm/ap_pipe/pull/6.diff","patch_url":"https://github.com/lsst-dm/ap_pipe/pull/6.patch","issue_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/6","number":6,"state":"open","locked":false,"title":"DM-11390: Plug prototype pipeline script into verify_ap framework","user":{"login":"mrawls","id":5659077,"avatar_url":"https://avatars2.githubusercontent.com/u/5659077?v=4","gravatar_id":"","url":"https://api.github.com/users/mrawls","html_url":"https://github.com/mrawls","followers_url":"https://api.github.com/users/mrawls/followers","following_url":"https://api.github.com/users/mrawls/following{/other_user}","gists_url":"https://api.github.com/users/mrawls/gists{/gist_id}","starred_url":"https://api.github.com/users/mrawls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mrawls/subscriptions","organizations_url":"https://api.github.com/users/mrawls/orgs","repos_url":"https://api.github.com/users/mrawls/repos","events_url":"https://api.github.com/users/mrawls/events{/privacy}","received_events_url":"https://api.github.com/users/mrawls/received_events","type":"User","site_admin":false},"body":"","created_at":"2017-09-21T02:35:02Z","updated_at":"2017-09-22T22:05:37Z","closed_at":null,"merged_at":null,"merge_commit_sha":"542eaa82f10e4904f92907c9f135ba9383e11991","assignee":null,"assignees":[],"requested_reviewers":[],"milestone":null,"commits_url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6/commits","review_comments_url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6/comments","review_comment_url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/comments{/number}","comments_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/6/comments","statuses_url":"https://api.github.com/repos/lsst-dm/ap_pipe/statuses/a3db926292cbc39123f2332ca2e51d9be4e2b0d9","head":{"label":"lsst-dm:tickets/DM-11390","ref":"tickets/DM-11390","sha":"a3db926292cbc39123f2332ca2e51d9be4e2b0d9","user":{"login":"lsst-dm","id":10968870,"avatar_url":"https://avatars1.githubusercontent.com/u/10968870?v=4","gravatar_id":"","url":"https://api.github.com/users/lsst-dm","html_url":"https://github.com/lsst-dm","followers_url":"https://api.github.com/users/lsst-dm/followers","following_url":"https://api.github.com/users/lsst-dm/following{/other_user}","gists_url":"https://api.github.com/users/lsst-dm/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst-dm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst-dm/subscriptions","organizations_url":"https://api.github.com/users/lsst-dm/orgs","repos_url":"https://api.github.com/users/lsst-dm/repos","events_url":"https://api.github.com/users/lsst-dm/events{/privacy}","received_events_url":"https://api.github.com/users/lsst-dm/received_events","type":"Organization","site_admin":false},"repo":{"id":77710130,"name":"ap_pipe","full_name":"lsst-dm/ap_pipe","owner":{"login":"lsst-dm","id":10968870,"avatar_url":"https://avatars1.githubusercontent.com/u/10968870?v=4","gravatar_id":"","url":"https://api.github.com/users/lsst-dm","html_url":"https://github.com/lsst-dm","followers_url":"https://api.github.com/users/lsst-dm/followers","following_url":"https://api.github.com/users/lsst-dm/following{/other_user}","gists_url":"https://api.github.com/users/lsst-dm/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst-dm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst-dm/subscriptions","organizations_url":"https://api.github.com/users/lsst-dm/orgs","repos_url":"https://api.github.com/users/lsst-dm/repos","events_url":"https://api.github.com/users/lsst-dm/events{/privacy}","received_events_url":"https://api.github.com/users/lsst-dm/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/lsst-dm/ap_pipe","description":"Using lsst/obs_decam to process raw HiTS data","fork":false,"url":"https://api.github.com/repos/lsst-dm/ap_pipe","forks_url":"https://api.github.com/repos/lsst-dm/ap_pipe/forks","keys_url":"https://api.github.com/repos/lsst-dm/ap_pipe/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lsst-dm/ap_pipe/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lsst-dm/ap_pipe/teams","hooks_url":"https://api.github.com/repos/lsst-dm/ap_pipe/hooks","issue_events_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/events{/number}","events_url":"https://api.github.com/repos/lsst-dm/ap_pipe/events","assignees_url":"https://api.github.com/repos/lsst-dm/ap_pipe/assignees{/user}","branches_url":"https://api.github.com/repos/lsst-dm/ap_pipe/branches{/branch}","tags_url":"https://api.github.com/repos/lsst-dm/ap_pipe/tags","blobs_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/refs{/sha}","trees_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lsst-dm/ap_pipe/statuses/{sha}","languages_url":"https://api.github.com/repos/lsst-dm/ap_pipe/languages","stargazers_url":"https://api.github.com/repos/lsst-dm/ap_pipe/stargazers","contributors_url":"https://api.github.com/repos/lsst-dm/ap_pipe/contributors","subscribers_url":"https://api.github.com/repos/lsst-dm/ap_pipe/subscribers","subscription_url":"https://api.github.com/repos/lsst-dm/ap_pipe/subscription","commits_url":"https://api.github.com/repos/lsst-dm/ap_pipe/commits{/sha}","git_commits_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/commits{/sha}","comments_url":"https://api.github.com/repos/lsst-dm/ap_pipe/comments{/number}","issue_comment_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/comments{/number}","contents_url":"https://api.github.com/repos/lsst-dm/ap_pipe/contents/{+path}","compare_url":"https://api.github.com/repos/lsst-dm/ap_pipe/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lsst-dm/ap_pipe/merges","archive_url":"https://api.github.com/repos/lsst-dm/ap_pipe/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lsst-dm/ap_pipe/downloads","issues_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues{/number}","pulls_url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls{/number}","milestones_url":"https://api.github.com/repos/lsst-dm/ap_pipe/milestones{/number}","notifications_url":"https://api.github.com/repos/lsst-dm/ap_pipe/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lsst-dm/ap_pipe/labels{/name}","releases_url":"https://api.github.com/repos/lsst-dm/ap_pipe/releases{/id}","deployments_url":"https://api.github.com/repos/lsst-dm/ap_pipe/deployments","created_at":"2016-12-30T20:58:57Z","updated_at":"2017-08-11T20:12:01Z","pushed_at":"2017-09-22T02:08:27Z","git_url":"git://github.com/lsst-dm/ap_pipe.git","ssh_url":"git@github.com:lsst-dm/ap_pipe.git","clone_url":"https://github.com/lsst-dm/ap_pipe.git","svn_url":"https://github.com/lsst-dm/ap_pipe","homepage":"","size":64,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":2,"forks":0,"open_issues":2,"watchers":0,"default_branch":"master"}},"base":{"label":"lsst-dm:master","ref":"master","sha":"2e9464a12c0187fa2ea27f8e4433a08c7a541947","user":{"login":"lsst-dm","id":10968870,"avatar_url":"https://avatars1.githubusercontent.com/u/10968870?v=4","gravatar_id":"","url":"https://api.github.com/users/lsst-dm","html_url":"https://github.com/lsst-dm","followers_url":"https://api.github.com/users/lsst-dm/followers","following_url":"https://api.github.com/users/lsst-dm/following{/other_user}","gists_url":"https://api.github.com/users/lsst-dm/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst-dm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst-dm/subscriptions","organizations_url":"https://api.github.com/users/lsst-dm/orgs","repos_url":"https://api.github.com/users/lsst-dm/repos","events_url":"https://api.github.com/users/lsst-dm/events{/privacy}","received_events_url":"https://api.github.com/users/lsst-dm/received_events","type":"Organization","site_admin":false},"repo":{"id":77710130,"name":"ap_pipe","full_name":"lsst-dm/ap_pipe","owner":{"login":"lsst-dm","id":10968870,"avatar_url":"https://avatars1.githubusercontent.com/u/10968870?v=4","gravatar_id":"","url":"https://api.github.com/users/lsst-dm","html_url":"https://github.com/lsst-dm","followers_url":"https://api.github.com/users/lsst-dm/followers","following_url":"https://api.github.com/users/lsst-dm/following{/other_user}","gists_url":"https://api.github.com/users/lsst-dm/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst-dm/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst-dm/subscriptions","organizations_url":"https://api.github.com/users/lsst-dm/orgs","repos_url":"https://api.github.com/users/lsst-dm/repos","events_url":"https://api.github.com/users/lsst-dm/events{/privacy}","received_events_url":"https://api.github.com/users/lsst-dm/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/lsst-dm/ap_pipe","description":"Using lsst/obs_decam to process raw HiTS data","fork":false,"url":"https://api.github.com/repos/lsst-dm/ap_pipe","forks_url":"https://api.github.com/repos/lsst-dm/ap_pipe/forks","keys_url":"https://api.github.com/repos/lsst-dm/ap_pipe/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lsst-dm/ap_pipe/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lsst-dm/ap_pipe/teams","hooks_url":"https://api.github.com/repos/lsst-dm/ap_pipe/hooks","issue_events_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/events{/number}","events_url":"https://api.github.com/repos/lsst-dm/ap_pipe/events","assignees_url":"https://api.github.com/repos/lsst-dm/ap_pipe/assignees{/user}","branches_url":"https://api.github.com/repos/lsst-dm/ap_pipe/branches{/branch}","tags_url":"https://api.github.com/repos/lsst-dm/ap_pipe/tags","blobs_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/refs{/sha}","trees_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lsst-dm/ap_pipe/statuses/{sha}","languages_url":"https://api.github.com/repos/lsst-dm/ap_pipe/languages","stargazers_url":"https://api.github.com/repos/lsst-dm/ap_pipe/stargazers","contributors_url":"https://api.github.com/repos/lsst-dm/ap_pipe/contributors","subscribers_url":"https://api.github.com/repos/lsst-dm/ap_pipe/subscribers","subscription_url":"https://api.github.com/repos/lsst-dm/ap_pipe/subscription","commits_url":"https://api.github.com/repos/lsst-dm/ap_pipe/commits{/sha}","git_commits_url":"https://api.github.com/repos/lsst-dm/ap_pipe/git/commits{/sha}","comments_url":"https://api.github.com/repos/lsst-dm/ap_pipe/comments{/number}","issue_comment_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/comments{/number}","contents_url":"https://api.github.com/repos/lsst-dm/ap_pipe/contents/{+path}","compare_url":"https://api.github.com/repos/lsst-dm/ap_pipe/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lsst-dm/ap_pipe/merges","archive_url":"https://api.github.com/repos/lsst-dm/ap_pipe/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lsst-dm/ap_pipe/downloads","issues_url":"https://api.github.com/repos/lsst-dm/ap_pipe/issues{/number}","pulls_url":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls{/number}","milestones_url":"https://api.github.com/repos/lsst-dm/ap_pipe/milestones{/number}","notifications_url":"https://api.github.com/repos/lsst-dm/ap_pipe/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lsst-dm/ap_pipe/labels{/name}","releases_url":"https://api.github.com/repos/lsst-dm/ap_pipe/releases{/id}","deployments_url":"https://api.github.com/repos/lsst-dm/ap_pipe/deployments","created_at":"2016-12-30T20:58:57Z","updated_at":"2017-08-11T20:12:01Z","pushed_at":"2017-09-22T02:08:27Z","git_url":"git://github.com/lsst-dm/ap_pipe.git","ssh_url":"git@github.com:lsst-dm/ap_pipe.git","clone_url":"https://github.com/lsst-dm/ap_pipe.git","svn_url":"https://github.com/lsst-dm/ap_pipe","homepage":"","size":64,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":2,"forks":0,"open_issues":2,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6"},"html":{"href":"https://github.com/lsst-dm/ap_pipe/pull/6"},"issue":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/6"},"comments":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/issues/6/comments"},"review_comments":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6/comments"},"review_comment":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/pulls/6/commits"},"statuses":{"href":"https://api.github.com/repos/lsst-dm/ap_pipe/statuses/a3db926292cbc39123f2332ca2e51d9be4e2b0d9"}},"author_association":"COLLABORATOR"}}
{"actor":{"display_login":"mrawls"}}