Slight revision of code for dumping connections #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
add links and change overview
change heading and restructuring
Adjust the images for top of page and update links
Fix links in docs
…length: 120 letters)
…example Include EI clustered network in examples index for nest-simulator
Co-authored-by: Hans Ekkehard Plesser <[email protected]>
Fix minor Sphinx warnings related to broken links
Fixed conductance based neuron model header files
Add warning concerning order of parameter creation and change in number of threads
Add note on parameter corrections for the AMAT2 model (Heiberg et al 2018)
Fix Izhikevich model equation on doc page
Co-authored-by: jessica-mitchell <[email protected]>
Co-authored-by: jessica-mitchell <[email protected]>
…diation_1691511208 Apply security best practices – Update `pre-commit` configuration
Update documentation for ```iaf_psc_exp```
Update VM Badge to v3.7
…sion Bump Ubuntu and Python versions for GitHub Actions workflows
|
@xavierotazuGDS This PR breaks some tests. This is due to the fact that these tests expect connections to be dumped in a specific order and the new algorithm outputs connections in a different. I am working on fixing this. |
|
@xavierotazuGDS I fixed the test now and also found that I had introduced a little bug, which I also have fixed now. |
xavierotazuGDS
pushed a commit
that referenced
this pull request
Jun 21, 2024
Revise connection_management.rst
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hi @xavierotazuGDS!
This PR slightly tidies up your code for dumping connections, and merges current master into your branch. Please merge this PR into your repo, then your PR nest#3160 towards NEST will be updated automatically.