{"action":"created","issue":{"url":"https://api.github.com/repos/ryanbressler/CloudForest/issues/56","repository_url":"https://api.github.com/repos/ryanbressler/CloudForest","labels_url":"https://api.github.com/repos/ryanbressler/CloudForest/issues/56/labels{/name}","comments_url":"https://api.github.com/repos/ryanbressler/CloudForest/issues/56/comments","events_url":"https://api.github.com/repos/ryanbressler/CloudForest/issues/56/events","html_url":"https://github.com/ryanbressler/CloudForest/pull/56","id":136746357,"number":56,"title":"Use local rand.Rand object in FeatureMatrix.BestSplitter","user":{"login":"Tasssadar","id":120108,"avatar_url":"https://avatars.githubusercontent.com/u/120108?v=3","gravatar_id":"","url":"https://api.github.com/users/Tasssadar","html_url":"https://github.com/Tasssadar","followers_url":"https://api.github.com/users/Tasssadar/followers","following_url":"https://api.github.com/users/Tasssadar/following{/other_user}","gists_url":"https://api.github.com/users/Tasssadar/gists{/gist_id}","starred_url":"https://api.github.com/users/Tasssadar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tasssadar/subscriptions","organizations_url":"https://api.github.com/users/Tasssadar/orgs","repos_url":"https://api.github.com/users/Tasssadar/repos","events_url":"https://api.github.com/users/Tasssadar/events{/privacy}","received_events_url":"https://api.github.com/users/Tasssadar/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2016-02-26T16:32:21Z","updated_at":"2016-05-10T10:27:33Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/ryanbressler/CloudForest/pulls/56","html_url":"https://github.com/ryanbressler/CloudForest/pull/56","diff_url":"https://github.com/ryanbressler/CloudForest/pull/56.diff","patch_url":"https://github.com/ryanbressler/CloudForest/pull/56.patch"},"body":"* When learning on many threads, there is a _lot_ of contention\r\n  on the mutex in global rand source for no real reason.\r\n* In my use case, results in speed up from 4m30s to 33s\r\n  on 64 core system (7.7x faster)\r\n\r\nppref graphs (**large images!**):\r\n* before: http://i.imgur.com/o0SxiM7.jpg\r\n* after: http://i.imgur.com/BeTErB8.jpg"},"comment":{"url":"https://api.github.com/repos/ryanbressler/CloudForest/issues/comments/218118740","html_url":"https://github.com/ryanbressler/CloudForest/pull/56#issuecomment-218118740","issue_url":"https://api.github.com/repos/ryanbressler/CloudForest/issues/56","id":218118740,"user":{"login":"Tasssadar","id":120108,"avatar_url":"https://avatars.githubusercontent.com/u/120108?v=3","gravatar_id":"","url":"https://api.github.com/users/Tasssadar","html_url":"https://github.com/Tasssadar","followers_url":"https://api.github.com/users/Tasssadar/followers","following_url":"https://api.github.com/users/Tasssadar/following{/other_user}","gists_url":"https://api.github.com/users/Tasssadar/gists{/gist_id}","starred_url":"https://api.github.com/users/Tasssadar/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/Tasssadar/subscriptions","organizations_url":"https://api.github.com/users/Tasssadar/orgs","repos_url":"https://api.github.com/users/Tasssadar/repos","events_url":"https://api.github.com/users/Tasssadar/events{/privacy}","received_events_url":"https://api.github.com/users/Tasssadar/received_events","type":"User","site_admin":false},"created_at":"2016-05-10T10:27:33Z","updated_at":"2016-05-10T10:27:33Z","body":"It now uses Rnd from BestSplitAllocs also in ShuffleCases (when you run Grow() with vet == true, it hits this contention point). The tests should now pass, too."}}