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.

Sign up for GitHub

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

Jump to bottom

Wrong instructions (example 2) #3202

Closed
cabriocat opened this issue May 2, 2024 · 1 comment · Fixed by #3275
Closed

Wrong instructions (example 2) #3202

cabriocat opened this issue May 2, 2024 · 1 comment · Fixed by #3275
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority stale Automatic marker for inactivity, please have another look here T: Maintenance Work to keep up the quality of the code and documentation.

Comments

@cabriocat
Copy link

cabriocat commented May 2, 2024 •
edited
Loading

It is not really a bug but a quick note: I am currently going through the example codes and noticed a mistake. In the example of a simple iaf neuron model with noise (second example) you noted that the first Poisson generator is 8kHZ. However, the code says 80kHz. This confused me for a while.

Instructions:

Third, the Poisson generator is configured using SetStatus, which expects
a list of node handles and a list of parameter dictionaries. We set the
Poisson generators to 8,000 Hz and 15,000 Hz, respectively. Note that we do
not need to set parameters for the neuron and the voltmeter, since they have
satisfactory defaults.

Code:
noise[0].rate = 80000.0 noise[1].rate = 15000.0
@jessica-mitchell jessica-mitchell added S: Normal Handle this with default priority T: Maintenance Work to keep up the quality of the code and documentation. I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) labels May 7, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 6, 2024

Issue automatically marked stale!

@github-actions github-actions bot added the stale Automatic marker for inactivity, please have another look here label Jul 6, 2024
jessica-mitchell added a commit to jessica-mitchell/nest-simulator that referenced this issue Aug 6, 2024
@jessica-mitchell
issue nest#3202
06f0372
@jessica-mitchell jessica-mitchell mentioned this issue Aug 6, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
I: No breaking change Previously written code will work as before, no one should note anything changing (aside the fix) S: Normal Handle this with default priority stale Automatic marker for inactivity, please have another look here T: Maintenance Work to keep up the quality of the code and documentation.
Projects
Documentation
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix minor issues in docs jessica-mitchell/nest-simulator
2 participants
@jessica-mitchell @cabriocat