Seed oddities in SpineML_2_BRAHMS_CL_weight.xsl

SpineML / SpineML_2_BRAHMS

Code generation scripts to run a SpineML neural model using BRAHMS on standard CPUs

http://spineml.github.io/simulators/BRAHMS/

4 stars 3 forks source link

Seed oddities in SpineML_2_BRAHMS_CL_weight.xsl #15

Closed sebjameswml closed 9 years ago

sebjameswml commented 9 years ago

See around line 270 of SpineML_2_BRAHMS_CL_weight.xsl:

// seed the rng:
zigset(&amp;this-&gt;rngData_BRAHMS, 1<xsl:value-of select=".//SMLNL:FixedProbabilityConnection/@seed"/>);
this-&gt;rngData_BRAHMS.seed = 123;

What's that '1' in front of the value of FixedProbabilityConnection and why is Data_BRAHMS.seed hardcoded to 123?

sebjameswml commented 9 years ago

Ok, the seed passed to zigset is separate from the seed set in rngData_BRAHMS.seed. I don't think it matters which of these is set to a user-supplied seed, as long as one of them changes as the user-supplied seed is changed. So, the existing code is perfectly acceptable, in my opinion.

sebjameswml commented 9 years ago

Closing; these seeds are ok.