{"action":"created","issue":{"url":"https://api.github.com/repos/josdejong/mathjs/issues/679","repository_url":"https://api.github.com/repos/josdejong/mathjs","labels_url":"https://api.github.com/repos/josdejong/mathjs/issues/679/labels{/name}","comments_url":"https://api.github.com/repos/josdejong/mathjs/issues/679/comments","events_url":"https://api.github.com/repos/josdejong/mathjs/issues/679/events","html_url":"https://github.com/josdejong/mathjs/pull/679","id":163056669,"number":679,"title":"add pickMultipleRandom function","user":{"login":"woylie","id":13847569,"avatar_url":"https://avatars.githubusercontent.com/u/13847569?v=3","gravatar_id":"","url":"https://api.github.com/users/woylie","html_url":"https://github.com/woylie","followers_url":"https://api.github.com/users/woylie/followers","following_url":"https://api.github.com/users/woylie/following{/other_user}","gists_url":"https://api.github.com/users/woylie/gists{/gist_id}","starred_url":"https://api.github.com/users/woylie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/woylie/subscriptions","organizations_url":"https://api.github.com/users/woylie/orgs","repos_url":"https://api.github.com/users/woylie/repos","events_url":"https://api.github.com/users/woylie/events{/privacy}","received_events_url":"https://api.github.com/users/woylie/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":9,"created_at":"2016-06-30T00:26:28Z","updated_at":"2016-07-18T01:00:54Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/josdejong/mathjs/pulls/679","html_url":"https://github.com/josdejong/mathjs/pull/679","diff_url":"https://github.com/josdejong/mathjs/pull/679.diff","patch_url":"https://github.com/josdejong/mathjs/pull/679.patch"},"body":"I added a pickMultipleRandom function modeled after pickRandom. I already wrote additional functions for weighted random picking, but since this is my first attempt at contributing to an open source project, I'd rather like to listen to your comments before putting it all out at once."},"comment":{"url":"https://api.github.com/repos/josdejong/mathjs/issues/comments/233214538","html_url":"https://github.com/josdejong/mathjs/pull/679#issuecomment-233214538","issue_url":"https://api.github.com/repos/josdejong/mathjs/issues/679","id":233214538,"user":{"login":"woylie","id":13847569,"avatar_url":"https://avatars.githubusercontent.com/u/13847569?v=3","gravatar_id":"","url":"https://api.github.com/users/woylie","html_url":"https://github.com/woylie","followers_url":"https://api.github.com/users/woylie/followers","following_url":"https://api.github.com/users/woylie/following{/other_user}","gists_url":"https://api.github.com/users/woylie/gists{/gist_id}","starred_url":"https://api.github.com/users/woylie/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/woylie/subscriptions","organizations_url":"https://api.github.com/users/woylie/orgs","repos_url":"https://api.github.com/users/woylie/repos","events_url":"https://api.github.com/users/woylie/events{/privacy}","received_events_url":"https://api.github.com/users/woylie/received_events","type":"User","site_admin":false},"created_at":"2016-07-18T01:00:53Z","updated_at":"2016-07-18T01:00:53Z","body":"I'm not sure whether it would make sense to include the weighted distribution in the available distributions. The weights array only relates to the given possibles array and can't be reused, in contrast to uniform and normal distributions. As soon as a weights array is passed, we just have to ignore the distribution of the distribution object. The option to pass weights doesn't really make sense inside a distribution object.\r\n\r\nI'm also wondering whether it would be a good idea to rewrite distribution.js so that the parameters (mean and sd for normal distribution) can be passed as well. This might be handy if we're going to add more distributions or statistical functions."}}
{"actor":{"display_login":"woylie"}}