-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Brian2GeNN not up to date with brian2 master #25
Comments
This is actually intended for the moment. To keep development time down, we froze brian2genn on a particular commit of brian2, but it will be updated for the final release of brian2. |
thanks. then we will proceed with the specific commits above for the comparison |
The heads of brian2genn and brian2 were again matched up 3 days ago. But the match is guaranteed to break more often in the coming weeks. |
thank you! but could you maybe refine your last post, i.e., which commits exactly (from the three repositories: brian2, genn, brian2genn) are supposed to be working for standard brian2 examples? |
Brian2: d82e7a4419e61d262126a96dc1bed35e37bac761 |
wow that was fast! have a nice weekend and thank you for the answer |
i would like to know whether more recent brian2genn/genn/brian2 commits are considered to be working. could you tell again the respective commit hashes (for *genn: of the development branches)? |
Pretty much nothing has happened since I commented 16 Oct. - those versions are your best bet atm. |
There is some problem with the brian2genn version cited above, so if interested in testing at this stage (15 Feb 2016) use: |
Brian2Genn is now working with the latest Brian 2 and GeNN master branches. |
Since this is indeed the case, I'll close this issue. |
When using the brian2genn interface (current develop branch) together with genn (current development branch) there were several problems related to code changes in Brian2.
For example: the class AttributeVariable does not exist anymore, the main_queue has sometimes an additional parameter, there were some changes to the clock structure, ...
Since we (@moritzaugustin and @Kwartke) want to benchmark our CUDA implementation against the GeNN simulator, a working brian2genn would ease the creation and execution of test suites.
Right now, we are using the following combination of repos (with slight modifications) -- thanks to Esin Yavuz for reporting the specific commits:
brian2: commit ac06993ac83101edf3fac3cec984f7ef161ea1a0
brian2genn: commit 5efbcc7
GeNN: commit 0621048c8ef0f081032f39303ce8846788bae9ef
The text was updated successfully, but these errors were encountered: