search

AStupidBear / SpikingNeuralNetworks.jl

Julia Spiking Neural Network Simulator
Other
37 stars 16 forks source link

Support for Adaptive Exponential Integrate and Fire Model. #23

Closed russelljjarvis closed 1 year ago

russelljjarvis commented 3 years ago

I love your Spiking Neural Networks package.

I was thinking it could support Adaptive Exponential models too, so I wrote code for doing that.

I also put a small note in the README.md for developers. Not sure if you would consider it helpful or not.

Russell.

AStupidBear commented 3 years ago

@jpsamaroo Any idea?

jpsamaroo commented 3 years ago

The README changes are pretty generic and apply to all Julia packages, so I don't think that change is necessary. New Julia users are generally expected to read the Julia and Pkg docs, which explains how to dev a package for development.

Can these commits be squashed? There's quite a lot of "noise".

jpsamaroo commented 3 years ago

Also, a stripped down version of the example (without Plots) should be added as a test, to ensure this code actually works.

russelljjarvis commented 3 years ago

That all sounds reasonable. I will create a stiped down test version and squash the commits. You are probably write about the documentation.

russelljjarvis commented 1 year ago

I closed this issue as adding AdExp is no longer a priority for me.