BrainGrid icon indicating copy to clipboard operation
BrainGrid copied to clipboard

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

Open stiber opened this issue 5 years ago • 0 comments

What kind of issue is this?

  • [ ] Bug report
  • [ ] Feature request
  • [ ] Question not answered in documentation
  • [X] 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 avatar Jun 07 '19 21:06 stiber