computation-thru-dynamics icon indicating copy to clipboard operation
computation-thru-dynamics copied to clipboard

Updated distribution of biases

Open mattgolub opened this issue 5 years ago • 2 comments

With this change, biases are ~Uniform(-bval, bval). Previously, biases were drawn from a Gaussian and were not centered at 0.

mattgolub avatar Jul 17 '20 01:07 mattgolub

Thanks @mattgolub! I've been aware of this, but there are two concerns about this. (1) This fix will change the noise distribution from gaussian to uniform distribution. Alternatively we can fix line 34 (i.e., removing -0.5). (2) The asymmetry might be intentional, in order to visualize an asymmetric line attractor manifold in the FixedPointFinder tutorial. @sussillo, could you comment on it?

youngju-jo avatar Jul 22 '20 23:07 youngju-jo

Thanks @mattgolub! I've been aware of this, but there are two concerns about this. (1) This fix will change the noise distribution from gaussian to uniform distribution. Alternatively we can fix line 34 (i.e., removing -0.5). (2) The asymmetry might be intentional, in order to visualize an asymmetric line attractor manifold in the FixedPointFinder tutorial. @sussillo, could you comment on it?

Definitely not intentional. Oops! I'd suggest getting rid of the 2.0 and the - 0.5. Thank you, -David

sussillo avatar Jul 22 '20 23:07 sussillo