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

Add NEST examples #801

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

HarshKhilawala
Copy link

@HarshKhilawala HarshKhilawala commented Jun 6, 2024

Reproduced same example of NEST - One Neuron using PyNN.

References:

@HarshKhilawala
Copy link
Author

@apdavison Requesting review and docstring changes

@HarshKhilawala
Copy link
Author

@apdavison Also Added One Neuron Noise Example. Please review and share docstring convention for PyNN

@HarshKhilawala HarshKhilawala changed the title Add single neuron NEST example Add NEST examples Jun 7, 2024
@HarshKhilawala
Copy link
Author

Added two neuron example in simple networks from NEST

@apdavison
Copy link
Member

Thanks for these @HarshKhilawala. These are rather simple examples, but they are helpful since they match examples in the NEST documentation.

@HarshKhilawala
Copy link
Author

HarshKhilawala commented Jun 13, 2024

@apdavison Here, my approach was to get familiarity with PyNN codebase so I considered implementing simple network from NEST with PyNN. But I do acknowledge my lack of prior experience in neuroscience.

Even with simple networks, I am facing some issues.

I have a separate PR for this where I request you to please review my code. (PR has changes linked to all files)

Refer to PR#802

@coveralls
Copy link

Coverage Status

coverage: 63.264% (-0.07%) from 63.329%
when pulling ce7a868 on HarshKhilawala:add-examples
into 1a1ce5c on NeuralEnsemble:master.

@HarshKhilawala
Copy link
Author

@apdavison Updated scripts. PTAL!

@HarshKhilawala
Copy link
Author

Ready to merge. Suggest additional changes if necessary

apdavison and others added 4 commits July 16, 2024 15:20
… PyNN version gives the same results as the PyNEST original.

This also sets the initial membrane potential to -70 mV, to match the NEST default.
- set initial membrane potential to -70 mV to match NEST default
- use a smaller time step (0.01 ms) otherwise Brian does not respond to the incoming spikes
- use separate Projections for the excitatory and inhibitory connections
…l to -70 mV; put both neurons into a single Population; plot both neurons on the same axes.
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.

3 participants