{"action":"created","issue":{"url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/issues/7","labels_url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/issues/7/labels{/name}","comments_url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/issues/7/comments","events_url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/issues/7/events","html_url":"https://github.com/lindahua/EmpiricalRisks.jl/pull/7","id":105055042,"number":7,"title":"Create mutating value! and predict! functions","user":{"login":"Evizero","id":10854026,"avatar_url":"https://avatars.githubusercontent.com/u/10854026?v=3","gravatar_id":"","url":"https://api.github.com/users/Evizero","html_url":"https://github.com/Evizero","followers_url":"https://api.github.com/users/Evizero/followers","following_url":"https://api.github.com/users/Evizero/following{/other_user}","gists_url":"https://api.github.com/users/Evizero/gists{/gist_id}","starred_url":"https://api.github.com/users/Evizero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Evizero/subscriptions","organizations_url":"https://api.github.com/users/Evizero/orgs","repos_url":"https://api.github.com/users/Evizero/repos","events_url":"https://api.github.com/users/Evizero/events{/privacy}","received_events_url":"https://api.github.com/users/Evizero/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-09-05T23:08:53Z","updated_at":"2015-09-06T08:23:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/pulls/7","html_url":"https://github.com/lindahua/EmpiricalRisks.jl/pull/7","diff_url":"https://github.com/lindahua/EmpiricalRisks.jl/pull/7.diff","patch_url":"https://github.com/lindahua/EmpiricalRisks.jl/pull/7.patch"},"body":"To reduce memory footprint as discussed in #6 \r\nI also included tests.\r\n\r\nAs a sidenote I also updated the readme badge for pkg.julialang.org to the new format (there was a link-breaking update)"},"comment":{"url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/issues/comments/138052036","html_url":"https://github.com/lindahua/EmpiricalRisks.jl/pull/7#issuecomment-138052036","issue_url":"https://api.github.com/repos/lindahua/EmpiricalRisks.jl/issues/7","id":138052036,"user":{"login":"Evizero","id":10854026,"avatar_url":"https://avatars.githubusercontent.com/u/10854026?v=3","gravatar_id":"","url":"https://api.github.com/users/Evizero","html_url":"https://github.com/Evizero","followers_url":"https://api.github.com/users/Evizero/followers","following_url":"https://api.github.com/users/Evizero/following{/other_user}","gists_url":"https://api.github.com/users/Evizero/gists{/gist_id}","starred_url":"https://api.github.com/users/Evizero/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Evizero/subscriptions","organizations_url":"https://api.github.com/users/Evizero/orgs","repos_url":"https://api.github.com/users/Evizero/repos","events_url":"https://api.github.com/users/Evizero/events{/privacy}","received_events_url":"https://api.github.com/users/Evizero/received_events","type":"User","site_admin":false},"created_at":"2015-09-06T08:23:44Z","updated_at":"2015-09-06T08:23:44Z","body":"Thank you for the tip, I didn't know about those functions.\r\n\r\nI did some benchmarking and concluded that for LinearPred and MvLinear Pred using these functions makes a lot of sense!\r\n\r\nHowever, for the AffinePred it is actually better using the loops. This is because the otherwise needed array view and broadcast! introduce notable overhead in terms of allocation and runtime"}}