{"action":"created","issue":{"url":"https://api.github.com/repos/brian-team/brian2/issues/576","labels_url":"https://api.github.com/repos/brian-team/brian2/issues/576/labels{/name}","comments_url":"https://api.github.com/repos/brian-team/brian2/issues/576/comments","events_url":"https://api.github.com/repos/brian-team/brian2/issues/576/events","html_url":"https://github.com/brian-team/brian2/pull/576","id":111661983,"number":576,"title":"[WIP] Fix Python spike queue","user":{"login":"mstimberg","id":1381982,"avatar_url":"https://avatars.githubusercontent.com/u/1381982?v=3","gravatar_id":"","url":"https://api.github.com/users/mstimberg","html_url":"https://github.com/mstimberg","followers_url":"https://api.github.com/users/mstimberg/followers","following_url":"https://api.github.com/users/mstimberg/following{/other_user}","gists_url":"https://api.github.com/users/mstimberg/gists{/gist_id}","starred_url":"https://api.github.com/users/mstimberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstimberg/subscriptions","organizations_url":"https://api.github.com/users/mstimberg/orgs","repos_url":"https://api.github.com/users/mstimberg/repos","events_url":"https://api.github.com/users/mstimberg/events{/privacy}","received_events_url":"https://api.github.com/users/mstimberg/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/brian-team/brian2/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":9,"created_at":"2015-10-15T17:01:01Z","updated_at":"2015-10-16T14:35:42Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/brian-team/brian2/pulls/576","html_url":"https://github.com/brian-team/brian2/pull/576","diff_url":"https://github.com/brian-team/brian2/pull/576.diff","patch_url":"https://github.com/brian-team/brian2/pull/576.patch"},"body":"I did not look into the issue in detail yet, but this branch includes an extended `test_transmission` test that fails with the Python spike queue.\r\n(BTW: This test only tests transmission with one-to-one synapses, we should at least have tests for many-to-one and one-to-many connections as well)."},"comment":{"url":"https://api.github.com/repos/brian-team/brian2/issues/comments/148734031","html_url":"https://github.com/brian-team/brian2/pull/576#issuecomment-148734031","issue_url":"https://api.github.com/repos/brian-team/brian2/issues/576","id":148734031,"user":{"login":"mstimberg","id":1381982,"avatar_url":"https://avatars.githubusercontent.com/u/1381982?v=3","gravatar_id":"","url":"https://api.github.com/users/mstimberg","html_url":"https://github.com/mstimberg","followers_url":"https://api.github.com/users/mstimberg/followers","following_url":"https://api.github.com/users/mstimberg/following{/other_user}","gists_url":"https://api.github.com/users/mstimberg/gists{/gist_id}","starred_url":"https://api.github.com/users/mstimberg/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mstimberg/subscriptions","organizations_url":"https://api.github.com/users/mstimberg/orgs","repos_url":"https://api.github.com/users/mstimberg/repos","events_url":"https://api.github.com/users/mstimberg/events{/privacy}","received_events_url":"https://api.github.com/users/mstimberg/received_events","type":"User","site_admin":false},"created_at":"2015-10-16T14:35:42Z","updated_at":"2015-10-16T14:35:42Z","body":"I did not do thorough benchmarking but testing with the CUDA example, I get (all times are for `synapses_pre` which includes the push but also the update itself):\r\nHomogeneous delays: ~6.45s\r\nHetereogeneous delays without pre-computation (but vectorized): 7.63s\r\nHetereogeneous delays with pre-computation: 6.85s\r\n\r\nFor comparison with the C++ spike queue: 6.05s\r\n\r\nSo it is faster, but the speed advantage has a negligible influence on the total run time. Do you maybe have more thorough benchmarking code laying around from the time when you looked at optimizing the C++ spike queue?\r\n"}}