Skip to content

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.
@HarshKhilawala
Copy link
Author

@apdavison Are there any more changes required? Can we get this merged?

@HarshKhilawala
Copy link
Author

@apdavison We never really got this merged. I wonder why my added examples aren’t reflected to PyNN. Can you please get it merged?

Also, Comment on my other PRs which needs necessary attention and changes.

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