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

Reduce primary_end_ marker for het connections #834

Merged
merged 3 commits into from Oct 12, 2017

Conversation

@sanjayankur31
Copy link
Contributor

sanjayankur31 commented Oct 1, 2017

When all connections of a particular synapse type are removed, the
entry is removed from HetConnector and the primary_end_ marker should
also be reduced by one.

Fixes #832

This may not be the best way of going about fixing the issue. Someone with more knowledge of the connection data structures should take a look and modify it keeping the code design in mind.

When all connections of a particular synapse type are removed, the
entry is removed from HetConnector and the primary_end_ marker should
also be reduced by one.

Fixes #832
Copy link

janhahne left a comment

@sanjayankur31 Thanks a lot for the fix! I send you a pull request with a regression test that ensures that this bug is fixed in a setup with additional gap junctions.

Once this is added I am all happy 👍

@jakobj This might be something that is important for future architecture as well. The new regression test will hopefully ensure that the bug is detected if it also applies there.

janhahne and others added 2 commits Oct 4, 2017
Add regressiontest
@jakobj
Copy link
Contributor

jakobj commented Oct 12, 2017

looks good to me 👍 will keep it in mind for future changes :)

@jakobj
jakobj approved these changes Oct 12, 2017
@terhorstd terhorstd merged commit ff33e38 into nest:master Oct 12, 2017
1 check passed
1 check passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
@sanjayankur31 sanjayankur31 deleted the sanjayankur31:issue-832 branch Nov 11, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

5 participants
You can’t perform that action at this time.