{"action":"created","issue":{"url":"https://api.github.com/repos/UDST/urbansim/issues/180","repository_url":"https://api.github.com/repos/UDST/urbansim","labels_url":"https://api.github.com/repos/UDST/urbansim/issues/180/labels{/name}","comments_url":"https://api.github.com/repos/UDST/urbansim/issues/180/comments","events_url":"https://api.github.com/repos/UDST/urbansim/issues/180/events","html_url":"https://github.com/UDST/urbansim/pull/180","id":189193124,"number":180,"title":"Fix sample rows no replace","user":{"login":"bridwell","id":5004800,"avatar_url":"https://avatars.githubusercontent.com/u/5004800?v=3","gravatar_id":"","url":"https://api.github.com/users/bridwell","html_url":"https://github.com/bridwell","followers_url":"https://api.github.com/users/bridwell/followers","following_url":"https://api.github.com/users/bridwell/following{/other_user}","gists_url":"https://api.github.com/users/bridwell/gists{/gist_id}","starred_url":"https://api.github.com/users/bridwell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/bridwell/subscriptions","organizations_url":"https://api.github.com/users/bridwell/orgs","repos_url":"https://api.github.com/users/bridwell/repos","events_url":"https://api.github.com/users/bridwell/events{/privacy}","received_events_url":"https://api.github.com/users/bridwell/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-11-14T18:49:49Z","updated_at":"2016-11-14T19:24:40Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/UDST/urbansim/pulls/180","html_url":"https://github.com/UDST/urbansim/pull/180","diff_url":"https://github.com/UDST/urbansim/pull/180.diff","patch_url":"https://github.com/UDST/urbansim/pull/180.patch"},"body":"This addresses feedback when the accounting total is not met (#178) and also providing probability distributions to influence the sample (#149).\r\n\r\nTo obtain feedback, set the return_status argument to True, and this will return both the sampled rows and the status. If False, then only the sampled rows will be returned. This is still the default, so as to not break any existing calls. \r\n\r\nTo provide sampling probabilities, set the prob_column argument to the name of the column in the data frame containing the weights or probabilities. These values will be normalized so they sum to 1, if they do not already.  \r\n\r\nThe logic for sampling with accounting but without replacement has been changed to hopefully be more robust. \r\n"},"comment":{"url":"https://api.github.com/repos/UDST/urbansim/issues/comments/260434957","html_url":"https://github.com/UDST/urbansim/pull/180#issuecomment-260434957","issue_url":"https://api.github.com/repos/UDST/urbansim/issues/180","id":260434957,"user":{"login":"coveralls","id":2354108,"avatar_url":"https://avatars.githubusercontent.com/u/2354108?v=3","gravatar_id":"","url":"https://api.github.com/users/coveralls","html_url":"https://github.com/coveralls","followers_url":"https://api.github.com/users/coveralls/followers","following_url":"https://api.github.com/users/coveralls/following{/other_user}","gists_url":"https://api.github.com/users/coveralls/gists{/gist_id}","starred_url":"https://api.github.com/users/coveralls/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/coveralls/subscriptions","organizations_url":"https://api.github.com/users/coveralls/orgs","repos_url":"https://api.github.com/users/coveralls/repos","events_url":"https://api.github.com/users/coveralls/events{/privacy}","received_events_url":"https://api.github.com/users/coveralls/received_events","type":"User","site_admin":false},"created_at":"2016-11-14T19:24:40Z","updated_at":"2016-11-14T19:24:40Z","body":"\n[![Coverage Status](https://coveralls.io/builds/8816892/badge)](https://coveralls.io/builds/8816892)\n\nCoverage increased (+0.1%) to 94.284% when pulling **1e58e3a4ee54fd1e18494aca1512d0ef8ddf4796 on AZMAG:fix_sample_rows_no_replace** into **caa6ab8afce408e22bc84a80a8e55cc6d552c845 on UDST:master**.\n"}}
{"actor":{"display_login":"coveralls"}}