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

Change name and comment for Cluster::m_nSynapticTransDelay; fix its initialization #256

Open
1 of 4 tasks
stiber opened this issue Jun 7, 2019 · 0 comments
Open
1 of 4 tasks

Comments

@stiber
Copy link
Contributor

stiber commented Jun 7, 2019

What kind of issue is this?

  • Bug report
  • Feature request
  • Question not answered in documentation
  • Cleanup needed

The static variable m_nSynapticTransDelay is actually the minimum synaptic transmission delay. It is the number of time steps that spikes can be buffered within a cluster before they need to be sent to their destinations in other clusters.

Besides name and comment, right now its value is hard-coded. It should be initialized based on the synaptic delay information for the current models.

        /**
         * The synaptic transmission delay, descretized into time steps
         */
        static int m_nSynapticTransDelay;

What is affected by this?

How do we replicate the issue/how would it work?

Expected behavior (i.e. solution or outline of what it would look like)

Other Comments

@stiber stiber added this to the Summer 2019 milestone Jun 7, 2019
@stiber stiber modified the milestones: Summer 2019, Fall 2019 Oct 14, 2019
@stiber stiber removed the BG/C++ label Feb 15, 2020
@stiber stiber modified the milestones: Fall 2019, Spring 2020 Feb 15, 2020
@stiber stiber modified the milestones: Spring 2020, Future Jul 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant