You seem to have used the version that was used for the consistency tests (that means with TimedArray implementations of the randomness). Not sure what the effect is on compile times due to curand etc.
And it seems you used a different script for brian2genn? In fact, you used one without TimedArray for brian2genn? You need to use the exact same script if you want to make things comparable.
Please don't add large files to the repository. This PR tried to add 80MB of files to this repository, which doubles the current repository size. This just makes things super slow. Most of the size comes from the arrays saved in all the results directory. Those you don't need at all, please don't add them to the repository. And the TimedArray implementations store the arrays in the static_arrays folders. Those alone take up 20MB.
TODOs:
How much work is it to repeat the merge for a script that doesn't use TimedArrays? If you think this is done in ~10min, please do that. If you have to manually merge everything again, just leave it as is. Then we will just not merge the branch.
Either way, please make sure that the script for brian2cuda and brian2genn are the same (either brian2cuda without TimedArray or brian2gen with TimedArray)
If you got rid of the TimedArrays, please create a new PR where only source files are added (no resulst folders). If recreating the script without TimedArray takes too much time, don't worry about this. We won't merge this PR in that case.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Related to #179