{"action":"created","issue":{"url":"https://api.github.com/repos/jsmeredith/EAVL/issues/21","repository_url":"https://api.github.com/repos/jsmeredith/EAVL","labels_url":"https://api.github.com/repos/jsmeredith/EAVL/issues/21/labels{/name}","comments_url":"https://api.github.com/repos/jsmeredith/EAVL/issues/21/comments","events_url":"https://api.github.com/repos/jsmeredith/EAVL/issues/21/events","html_url":"https://github.com/jsmeredith/EAVL/pull/21","id":131222440,"number":21,"title":"OpenMp parallel scheduler","user":{"login":"applekey","id":2405077,"avatar_url":"https://avatars.githubusercontent.com/u/2405077?v=3","gravatar_id":"","url":"https://api.github.com/users/applekey","html_url":"https://github.com/applekey","followers_url":"https://api.github.com/users/applekey/followers","following_url":"https://api.github.com/users/applekey/following{/other_user}","gists_url":"https://api.github.com/users/applekey/gists{/gist_id}","starred_url":"https://api.github.com/users/applekey/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/applekey/subscriptions","organizations_url":"https://api.github.com/users/applekey/orgs","repos_url":"https://api.github.com/users/applekey/repos","events_url":"https://api.github.com/users/applekey/events{/privacy}","received_events_url":"https://api.github.com/users/applekey/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2016-02-04T02:42:55Z","updated_at":"2016-02-05T18:24:50Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/jsmeredith/EAVL/pulls/21","html_url":"https://github.com/jsmeredith/EAVL/pull/21","diff_url":"https://github.com/jsmeredith/EAVL/pull/21.diff","patch_url":"https://github.com/jsmeredith/EAVL/pull/21.patch"},"body":"The default parallel for scheduler for openmp seems to be static and its slow.  Roba was previously benchmarking her volume rendering code that used a map operator and found that it got abysmal speedups, 1.5x for 2 cores, 2x for 3 cores...etc etc and this was the issue.  The dynamic scheduler may not be appropriate for all circumstances, so this pull request may not be necessary, but it highlights a question, should the default scheduler be kept at static? "},"comment":{"url":"https://api.github.com/repos/jsmeredith/EAVL/issues/comments/180483486","html_url":"https://github.com/jsmeredith/EAVL/pull/21#issuecomment-180483486","issue_url":"https://api.github.com/repos/jsmeredith/EAVL/issues/21","id":180483486,"user":{"login":"mclarsen","id":4038129,"avatar_url":"https://avatars.githubusercontent.com/u/4038129?v=3","gravatar_id":"","url":"https://api.github.com/users/mclarsen","html_url":"https://github.com/mclarsen","followers_url":"https://api.github.com/users/mclarsen/followers","following_url":"https://api.github.com/users/mclarsen/following{/other_user}","gists_url":"https://api.github.com/users/mclarsen/gists{/gist_id}","starred_url":"https://api.github.com/users/mclarsen/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mclarsen/subscriptions","organizations_url":"https://api.github.com/users/mclarsen/orgs","repos_url":"https://api.github.com/users/mclarsen/repos","events_url":"https://api.github.com/users/mclarsen/events{/privacy}","received_events_url":"https://api.github.com/users/mclarsen/received_events","type":"User","site_admin":false},"created_at":"2016-02-05T18:24:50Z","updated_at":"2016-02-05T18:24:50Z","body":"Well, I think that this particular problem maybe memory bound. As more \nthreads do work, the memory systems get saturated and scaling suffers.\n\n-Matt\n\nOn 2016-02-05 10:17, Vincent Chen wrote:\n> Sorry, had some previous run result data jumbled in that graph.\n> \n> https://plot.ly/~applekey/7/static-dynamic-guided-auto/ [1]\n> \n> --\n> Reply to this email directly or view it on GitHub [2].\n> \n> \t*\n> \n> Links:\n> ------\n> [1] https://plot.ly/%7Eapplekey/7/static-dynamic-guided-auto/\n> [2] https://github.com/jsmeredith/EAVL/pull/21#issuecomment-180479594\n\n"}}