Skip to content

Add NumbaLIF neuron model #86

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

Closed
wants to merge 1 commit into from
Closed

Add NumbaLIF neuron model #86

wants to merge 1 commit into from

Conversation

drasmuss
Copy link
Member

Migrated from nengo/nengo#1482

I also reworked the setup.py extra_requires slightly, since I needed to add in numba and there wasn't an obvious place to put it. Could revert this if we want to keep requirements separated by different subcomponents (e.g. one for deeplearning, one for numbalif). But I figured it'd be ok to lump everything under "optional", and that's familiar for people working with other nengo projects.

Based on #85, as there are some fixes in that branch needed to get nengo-extras tests passing.

A numba-accelerated drop-in replacement for nengo.LIF.

Also reorganized the setup.py extra_requires slightly
to align with other nengo projects.
@hunse
Copy link
Collaborator

hunse commented May 24, 2019

Don't know why this didn't close automatically, it's merged and the merged branch is up to date with master.

@hunse hunse closed this May 24, 2019
@hunse hunse deleted the numba-lif branch May 24, 2019 17:44
@tbekolay
Copy link
Member

It's because the base branch for this PR isn't master, it's realtime

@hunse
Copy link
Collaborator

hunse commented May 24, 2019

Ahh, and that's why it wouldn't let me delete the realtime branch earlier. I just thought this was rebased onto realtime, not a PR into realtime.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants