Skip to content
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

Various codegen improvements #101

Merged
merged 3 commits into from
Sep 5, 2013
Merged

Various codegen improvements #101

merged 3 commits into from
Sep 5, 2013

Conversation

thesamovar
Copy link
Member

Don't merge yet, for the moment just using this pull request to track the work in this branch.

For a start, resolves #100.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling e3f1854 on codegen_improvement into ed5ec94 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 6a1546f on codegen_improvement into ed5ec94 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a1cf422 on codegen_improvement into ed5ec94 on master.

@thesamovar
Copy link
Member Author

So actually there are various things I'd like to do in this branch, including possibly resolving all of issues #104, #99, #96, #80 and #63 - see recently added comments in those issues. I think with all these done we'll have a much nicer templating system with simpler, more understandable Brian code, and more flexible to work with standalone as well as runtime.

@thesamovar
Copy link
Member Author

I think I'm going to put future codegen improvements in the devices2 branch. Now that we plan to merge that branch regularly into master, maybe we should just merge this and delete the branch? @mstimberg want to go ahead and do that?

@mstimberg
Copy link
Member

Looks good to me, merging now.

mstimberg pushed a commit that referenced this pull request Sep 5, 2013
@mstimberg mstimberg merged commit 36a55c0 into master Sep 5, 2013
@mstimberg mstimberg deleted the codegen_improvement branch September 5, 2013 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make CodeObject Nameable
3 participants