{"action":"created","comment":{"url":"https://api.github.com/repos/lsst/sims_catUtils/pulls/comments/74546599","id":74546599,"diff_hunk":"@@ -309,11 +314,14 @@ def get_pointings(self, ra, dec, bandpass=('u', 'g', 'r', 'i', 'z', 'y'), expMJD\n \n         return obs_groups_out\n \n-    def _get_query_from_group(self, grp, chunk_size):\n+    def _get_query_from_group(self, grp, chunk_size, lc_per_field=None):\n         \"\"\"","path":"python/lsst/sims/catUtils/utils/LightCurveGenerator.py","position":18,"original_position":18,"commit_id":"e2538f899f40047d6b5ae2fb7feb50b8f69ef250","original_commit_id":"e2538f899f40047d6b5ae2fb7feb50b8f69ef250","user":{"login":"jbkalmbach","id":4755090,"avatar_url":"https://avatars.githubusercontent.com/u/4755090?v=3","gravatar_id":"","url":"https://api.github.com/users/jbkalmbach","html_url":"https://github.com/jbkalmbach","followers_url":"https://api.github.com/users/jbkalmbach/followers","following_url":"https://api.github.com/users/jbkalmbach/following{/other_user}","gists_url":"https://api.github.com/users/jbkalmbach/gists{/gist_id}","starred_url":"https://api.github.com/users/jbkalmbach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbkalmbach/subscriptions","organizations_url":"https://api.github.com/users/jbkalmbach/orgs","repos_url":"https://api.github.com/users/jbkalmbach/repos","events_url":"https://api.github.com/users/jbkalmbach/events{/privacy}","received_events_url":"https://api.github.com/users/jbkalmbach/received_events","type":"User","site_admin":false},"body":"For consistency I think we should properly document all input and output parameters here as we have elsewhere.","created_at":"2016-08-12T06:32:17Z","updated_at":"2016-08-12T06:32:17Z","html_url":"https://github.com/lsst/sims_catUtils/pull/63#discussion_r74546599","pull_request_url":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63","_links":{"self":{"href":"https://api.github.com/repos/lsst/sims_catUtils/pulls/comments/74546599"},"html":{"href":"https://github.com/lsst/sims_catUtils/pull/63#discussion_r74546599"},"pull_request":{"href":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63"}}},"pull_request":{"url":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63","id":79283634,"html_url":"https://github.com/lsst/sims_catUtils/pull/63","diff_url":"https://github.com/lsst/sims_catUtils/pull/63.diff","patch_url":"https://github.com/lsst/sims_catUtils/pull/63.patch","issue_url":"https://api.github.com/repos/lsst/sims_catUtils/issues/63","number":63,"state":"open","locked":false,"title":"Feature/sim 2068/lc functionality","user":{"login":"danielsf","id":1682854,"avatar_url":"https://avatars.githubusercontent.com/u/1682854?v=3","gravatar_id":"","url":"https://api.github.com/users/danielsf","html_url":"https://github.com/danielsf","followers_url":"https://api.github.com/users/danielsf/followers","following_url":"https://api.github.com/users/danielsf/following{/other_user}","gists_url":"https://api.github.com/users/danielsf/gists{/gist_id}","starred_url":"https://api.github.com/users/danielsf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/danielsf/subscriptions","organizations_url":"https://api.github.com/users/danielsf/orgs","repos_url":"https://api.github.com/users/danielsf/repos","events_url":"https://api.github.com/users/danielsf/events{/privacy}","received_events_url":"https://api.github.com/users/danielsf/received_events","type":"User","site_admin":false},"body":"This pull request does a few things\r\n\r\n1) makes it possible to request a limited number of rows from our \"high tech\" CatalogDBObjects (those that use htmid to do the spatial query)\r\n\r\n2) makes it possible to request a limited number of light curves from the LightCurveGenerator\r\n\r\n3) moves testSN.py into the tests/ directory so that it will automatically be run by scons.  I had to add a conditional to ensure that the unit tests are skipped on Jenkins, where, for some reason, the environment expected by astropy does not properly load.","created_at":"2016-07-28T23:08:16Z","updated_at":"2016-08-12T06:32:17Z","closed_at":null,"merged_at":null,"merge_commit_sha":"f49743bf85b03b4c739342a153126f07af8158fc","assignee":{"login":"jbkalmbach","id":4755090,"avatar_url":"https://avatars.githubusercontent.com/u/4755090?v=3","gravatar_id":"","url":"https://api.github.com/users/jbkalmbach","html_url":"https://github.com/jbkalmbach","followers_url":"https://api.github.com/users/jbkalmbach/followers","following_url":"https://api.github.com/users/jbkalmbach/following{/other_user}","gists_url":"https://api.github.com/users/jbkalmbach/gists{/gist_id}","starred_url":"https://api.github.com/users/jbkalmbach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbkalmbach/subscriptions","organizations_url":"https://api.github.com/users/jbkalmbach/orgs","repos_url":"https://api.github.com/users/jbkalmbach/repos","events_url":"https://api.github.com/users/jbkalmbach/events{/privacy}","received_events_url":"https://api.github.com/users/jbkalmbach/received_events","type":"User","site_admin":false},"assignees":[{"login":"jbkalmbach","id":4755090,"avatar_url":"https://avatars.githubusercontent.com/u/4755090?v=3","gravatar_id":"","url":"https://api.github.com/users/jbkalmbach","html_url":"https://github.com/jbkalmbach","followers_url":"https://api.github.com/users/jbkalmbach/followers","following_url":"https://api.github.com/users/jbkalmbach/following{/other_user}","gists_url":"https://api.github.com/users/jbkalmbach/gists{/gist_id}","starred_url":"https://api.github.com/users/jbkalmbach/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jbkalmbach/subscriptions","organizations_url":"https://api.github.com/users/jbkalmbach/orgs","repos_url":"https://api.github.com/users/jbkalmbach/repos","events_url":"https://api.github.com/users/jbkalmbach/events{/privacy}","received_events_url":"https://api.github.com/users/jbkalmbach/received_events","type":"User","site_admin":false}],"milestone":null,"commits_url":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63/commits","review_comments_url":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63/comments","review_comment_url":"https://api.github.com/repos/lsst/sims_catUtils/pulls/comments{/number}","comments_url":"https://api.github.com/repos/lsst/sims_catUtils/issues/63/comments","statuses_url":"https://api.github.com/repos/lsst/sims_catUtils/statuses/e2538f899f40047d6b5ae2fb7feb50b8f69ef250","head":{"label":"lsst:feature/SIM-2068/lc_functionality","ref":"feature/SIM-2068/lc_functionality","sha":"e2538f899f40047d6b5ae2fb7feb50b8f69ef250","user":{"login":"lsst","id":1134504,"avatar_url":"https://avatars.githubusercontent.com/u/1134504?v=3","gravatar_id":"","url":"https://api.github.com/users/lsst","html_url":"https://github.com/lsst","followers_url":"https://api.github.com/users/lsst/followers","following_url":"https://api.github.com/users/lsst/following{/other_user}","gists_url":"https://api.github.com/users/lsst/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst/subscriptions","organizations_url":"https://api.github.com/users/lsst/orgs","repos_url":"https://api.github.com/users/lsst/repos","events_url":"https://api.github.com/users/lsst/events{/privacy}","received_events_url":"https://api.github.com/users/lsst/received_events","type":"Organization","site_admin":false},"repo":{"id":34668130,"name":"sims_catUtils","full_name":"lsst/sims_catUtils","owner":{"login":"lsst","id":1134504,"avatar_url":"https://avatars.githubusercontent.com/u/1134504?v=3","gravatar_id":"","url":"https://api.github.com/users/lsst","html_url":"https://github.com/lsst","followers_url":"https://api.github.com/users/lsst/followers","following_url":"https://api.github.com/users/lsst/following{/other_user}","gists_url":"https://api.github.com/users/lsst/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst/subscriptions","organizations_url":"https://api.github.com/users/lsst/orgs","repos_url":"https://api.github.com/users/lsst/repos","events_url":"https://api.github.com/users/lsst/events{/privacy}","received_events_url":"https://api.github.com/users/lsst/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/lsst/sims_catUtils","description":"LSST Simulations package for catalog utilities","fork":false,"url":"https://api.github.com/repos/lsst/sims_catUtils","forks_url":"https://api.github.com/repos/lsst/sims_catUtils/forks","keys_url":"https://api.github.com/repos/lsst/sims_catUtils/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lsst/sims_catUtils/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lsst/sims_catUtils/teams","hooks_url":"https://api.github.com/repos/lsst/sims_catUtils/hooks","issue_events_url":"https://api.github.com/repos/lsst/sims_catUtils/issues/events{/number}","events_url":"https://api.github.com/repos/lsst/sims_catUtils/events","assignees_url":"https://api.github.com/repos/lsst/sims_catUtils/assignees{/user}","branches_url":"https://api.github.com/repos/lsst/sims_catUtils/branches{/branch}","tags_url":"https://api.github.com/repos/lsst/sims_catUtils/tags","blobs_url":"https://api.github.com/repos/lsst/sims_catUtils/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lsst/sims_catUtils/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lsst/sims_catUtils/git/refs{/sha}","trees_url":"https://api.github.com/repos/lsst/sims_catUtils/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lsst/sims_catUtils/statuses/{sha}","languages_url":"https://api.github.com/repos/lsst/sims_catUtils/languages","stargazers_url":"https://api.github.com/repos/lsst/sims_catUtils/stargazers","contributors_url":"https://api.github.com/repos/lsst/sims_catUtils/contributors","subscribers_url":"https://api.github.com/repos/lsst/sims_catUtils/subscribers","subscription_url":"https://api.github.com/repos/lsst/sims_catUtils/subscription","commits_url":"https://api.github.com/repos/lsst/sims_catUtils/commits{/sha}","git_commits_url":"https://api.github.com/repos/lsst/sims_catUtils/git/commits{/sha}","comments_url":"https://api.github.com/repos/lsst/sims_catUtils/comments{/number}","issue_comment_url":"https://api.github.com/repos/lsst/sims_catUtils/issues/comments{/number}","contents_url":"https://api.github.com/repos/lsst/sims_catUtils/contents/{+path}","compare_url":"https://api.github.com/repos/lsst/sims_catUtils/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lsst/sims_catUtils/merges","archive_url":"https://api.github.com/repos/lsst/sims_catUtils/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lsst/sims_catUtils/downloads","issues_url":"https://api.github.com/repos/lsst/sims_catUtils/issues{/number}","pulls_url":"https://api.github.com/repos/lsst/sims_catUtils/pulls{/number}","milestones_url":"https://api.github.com/repos/lsst/sims_catUtils/milestones{/number}","notifications_url":"https://api.github.com/repos/lsst/sims_catUtils/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lsst/sims_catUtils/labels{/name}","releases_url":"https://api.github.com/repos/lsst/sims_catUtils/releases{/id}","deployments_url":"https://api.github.com/repos/lsst/sims_catUtils/deployments","created_at":"2015-04-27T13:25:00Z","updated_at":"2016-04-19T16:14:23Z","pushed_at":"2016-08-10T05:41:10Z","git_url":"git://github.com/lsst/sims_catUtils.git","ssh_url":"git@github.com:lsst/sims_catUtils.git","clone_url":"https://github.com/lsst/sims_catUtils.git","svn_url":"https://github.com/lsst/sims_catUtils","homepage":null,"size":9048,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":9,"forks":3,"open_issues":9,"watchers":0,"default_branch":"master"}},"base":{"label":"lsst:master","ref":"master","sha":"883ed914d5bbba38fdd59faf2d82f26cb0d97a6e","user":{"login":"lsst","id":1134504,"avatar_url":"https://avatars.githubusercontent.com/u/1134504?v=3","gravatar_id":"","url":"https://api.github.com/users/lsst","html_url":"https://github.com/lsst","followers_url":"https://api.github.com/users/lsst/followers","following_url":"https://api.github.com/users/lsst/following{/other_user}","gists_url":"https://api.github.com/users/lsst/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst/subscriptions","organizations_url":"https://api.github.com/users/lsst/orgs","repos_url":"https://api.github.com/users/lsst/repos","events_url":"https://api.github.com/users/lsst/events{/privacy}","received_events_url":"https://api.github.com/users/lsst/received_events","type":"Organization","site_admin":false},"repo":{"id":34668130,"name":"sims_catUtils","full_name":"lsst/sims_catUtils","owner":{"login":"lsst","id":1134504,"avatar_url":"https://avatars.githubusercontent.com/u/1134504?v=3","gravatar_id":"","url":"https://api.github.com/users/lsst","html_url":"https://github.com/lsst","followers_url":"https://api.github.com/users/lsst/followers","following_url":"https://api.github.com/users/lsst/following{/other_user}","gists_url":"https://api.github.com/users/lsst/gists{/gist_id}","starred_url":"https://api.github.com/users/lsst/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/lsst/subscriptions","organizations_url":"https://api.github.com/users/lsst/orgs","repos_url":"https://api.github.com/users/lsst/repos","events_url":"https://api.github.com/users/lsst/events{/privacy}","received_events_url":"https://api.github.com/users/lsst/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/lsst/sims_catUtils","description":"LSST Simulations package for catalog utilities","fork":false,"url":"https://api.github.com/repos/lsst/sims_catUtils","forks_url":"https://api.github.com/repos/lsst/sims_catUtils/forks","keys_url":"https://api.github.com/repos/lsst/sims_catUtils/keys{/key_id}","collaborators_url":"https://api.github.com/repos/lsst/sims_catUtils/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/lsst/sims_catUtils/teams","hooks_url":"https://api.github.com/repos/lsst/sims_catUtils/hooks","issue_events_url":"https://api.github.com/repos/lsst/sims_catUtils/issues/events{/number}","events_url":"https://api.github.com/repos/lsst/sims_catUtils/events","assignees_url":"https://api.github.com/repos/lsst/sims_catUtils/assignees{/user}","branches_url":"https://api.github.com/repos/lsst/sims_catUtils/branches{/branch}","tags_url":"https://api.github.com/repos/lsst/sims_catUtils/tags","blobs_url":"https://api.github.com/repos/lsst/sims_catUtils/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/lsst/sims_catUtils/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/lsst/sims_catUtils/git/refs{/sha}","trees_url":"https://api.github.com/repos/lsst/sims_catUtils/git/trees{/sha}","statuses_url":"https://api.github.com/repos/lsst/sims_catUtils/statuses/{sha}","languages_url":"https://api.github.com/repos/lsst/sims_catUtils/languages","stargazers_url":"https://api.github.com/repos/lsst/sims_catUtils/stargazers","contributors_url":"https://api.github.com/repos/lsst/sims_catUtils/contributors","subscribers_url":"https://api.github.com/repos/lsst/sims_catUtils/subscribers","subscription_url":"https://api.github.com/repos/lsst/sims_catUtils/subscription","commits_url":"https://api.github.com/repos/lsst/sims_catUtils/commits{/sha}","git_commits_url":"https://api.github.com/repos/lsst/sims_catUtils/git/commits{/sha}","comments_url":"https://api.github.com/repos/lsst/sims_catUtils/comments{/number}","issue_comment_url":"https://api.github.com/repos/lsst/sims_catUtils/issues/comments{/number}","contents_url":"https://api.github.com/repos/lsst/sims_catUtils/contents/{+path}","compare_url":"https://api.github.com/repos/lsst/sims_catUtils/compare/{base}...{head}","merges_url":"https://api.github.com/repos/lsst/sims_catUtils/merges","archive_url":"https://api.github.com/repos/lsst/sims_catUtils/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/lsst/sims_catUtils/downloads","issues_url":"https://api.github.com/repos/lsst/sims_catUtils/issues{/number}","pulls_url":"https://api.github.com/repos/lsst/sims_catUtils/pulls{/number}","milestones_url":"https://api.github.com/repos/lsst/sims_catUtils/milestones{/number}","notifications_url":"https://api.github.com/repos/lsst/sims_catUtils/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/lsst/sims_catUtils/labels{/name}","releases_url":"https://api.github.com/repos/lsst/sims_catUtils/releases{/id}","deployments_url":"https://api.github.com/repos/lsst/sims_catUtils/deployments","created_at":"2015-04-27T13:25:00Z","updated_at":"2016-04-19T16:14:23Z","pushed_at":"2016-08-10T05:41:10Z","git_url":"git://github.com/lsst/sims_catUtils.git","ssh_url":"git@github.com:lsst/sims_catUtils.git","clone_url":"https://github.com/lsst/sims_catUtils.git","svn_url":"https://github.com/lsst/sims_catUtils","homepage":null,"size":9048,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":3,"mirror_url":null,"open_issues_count":9,"forks":3,"open_issues":9,"watchers":0,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63"},"html":{"href":"https://github.com/lsst/sims_catUtils/pull/63"},"issue":{"href":"https://api.github.com/repos/lsst/sims_catUtils/issues/63"},"comments":{"href":"https://api.github.com/repos/lsst/sims_catUtils/issues/63/comments"},"review_comments":{"href":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63/comments"},"review_comment":{"href":"https://api.github.com/repos/lsst/sims_catUtils/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/lsst/sims_catUtils/pulls/63/commits"},"statuses":{"href":"https://api.github.com/repos/lsst/sims_catUtils/statuses/e2538f899f40047d6b5ae2fb7feb50b8f69ef250"}}}}
{"actor":{"display_login":"jbkalmbach"}}