{"action":"created","issue":{"url":"https://api.github.com/repos/nest/nest-simulator/issues/289","repository_url":"https://api.github.com/repos/nest/nest-simulator","labels_url":"https://api.github.com/repos/nest/nest-simulator/issues/289/labels{/name}","comments_url":"https://api.github.com/repos/nest/nest-simulator/issues/289/comments","events_url":"https://api.github.com/repos/nest/nest-simulator/issues/289/events","html_url":"https://github.com/nest/nest-simulator/pull/289","id":144376196,"number":289,"title":"hpc_benchmark fix and cleanup","user":{"login":"jakobj","id":1562742,"avatar_url":"https://avatars.githubusercontent.com/u/1562742?v=3","gravatar_id":"","url":"https://api.github.com/users/jakobj","html_url":"https://github.com/jakobj","followers_url":"https://api.github.com/users/jakobj/followers","following_url":"https://api.github.com/users/jakobj/following{/other_user}","gists_url":"https://api.github.com/users/jakobj/gists{/gist_id}","starred_url":"https://api.github.com/users/jakobj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakobj/subscriptions","organizations_url":"https://api.github.com/users/jakobj/orgs","repos_url":"https://api.github.com/users/jakobj/repos","events_url":"https://api.github.com/users/jakobj/events{/privacy}","received_events_url":"https://api.github.com/users/jakobj/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2016-03-29T20:20:23Z","updated_at":"2016-03-29T20:49:44Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nest/nest-simulator/pulls/289","html_url":"https://github.com/nest/nest-simulator/pull/289","diff_url":"https://github.com/nest/nest-simulator/pull/289.diff","patch_url":"https://github.com/nest/nest-simulator/pull/289.patch"},"body":"this PR fixes an issue that lead to poor scaling pf the `hpc_benchmark.sli` example. the issue was the use of `[E_from E_to] Range` in randomizing membrane potentials of neurons. this would allocate memory for an array containing the gids of the entire network and loop over this list on every process(!), which quickly becomes an issue for large scale benchmarks (in terms of time and memory). it is replaced by `E_net GetLocalNodes`.\r\nin addition I have removed output via `=` which happens on every process. all output should happen via the logger, which restricts the number of processes that generate output.\r\nI have added a rate calculation function, which was shamelessly copied from `brunel-2000.sli`.\r\nminor changes in this PR:\r\n\r\n- removed some whitespace at end of lines\r\n- removed some blank lines\r\n- fixed indentation\r\n- replaced [unit] with (unit)\r\n- minor addition to documentation\r\n- changed default scale to 2 (0.2 was pointless since this amounts to a five-times fully connected network)"},"comment":{"url":"https://api.github.com/repos/nest/nest-simulator/issues/comments/203095630","html_url":"https://github.com/nest/nest-simulator/pull/289#issuecomment-203095630","issue_url":"https://api.github.com/repos/nest/nest-simulator/issues/289","id":203095630,"user":{"login":"jakobj","id":1562742,"avatar_url":"https://avatars.githubusercontent.com/u/1562742?v=3","gravatar_id":"","url":"https://api.github.com/users/jakobj","html_url":"https://github.com/jakobj","followers_url":"https://api.github.com/users/jakobj/followers","following_url":"https://api.github.com/users/jakobj/following{/other_user}","gists_url":"https://api.github.com/users/jakobj/gists{/gist_id}","starred_url":"https://api.github.com/users/jakobj/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jakobj/subscriptions","organizations_url":"https://api.github.com/users/jakobj/orgs","repos_url":"https://api.github.com/users/jakobj/repos","events_url":"https://api.github.com/users/jakobj/events{/privacy}","received_events_url":"https://api.github.com/users/jakobj/received_events","type":"User","site_admin":false},"created_at":"2016-03-29T20:49:44Z","updated_at":"2016-03-29T20:49:44Z","body":"oh, right, forgot: I nominate @mschmidt87 and @suku248 as reviewers."}}