Hi list, I recently find that the noise seed block needs negative seed for gaussian and uniform distributions. The commit 0d6871b3 has provided documentation for this and also changed the seed in noise seed from positive value to 0 to avoid misleading. However, the seed value in channel model block hasn't been updated. And in some python scripts, we use `randint(0, 100000)` as the seed. This is misleading and doesn't give what we want. I attach a patch to address the issue.
on 2012-11-02 13:07
on 2012-11-02 18:16
On Fri, Nov 2, 2012 at 5:06 AM, Alick Zhao <alick9188@gmail.com> wrote: > I recently find that the noise seed block needs negative seed for > gaussian and uniform distributions. The commit 0d6871b3 has provided > documentation for this and also changed the seed in noise seed from > positive value to 0 to avoid misleading. However, the seed value in > channel model block hasn't been updated. And in some python scripts, > we use `randint(0, 100000)` as the seed. This is misleading and doesn't > give what we want. > > I attach a patch to address the issue. > Thanks, this has been applied. Johnathan
Please log in before posting. Registration is free and takes only a minute.
Existing account
(Switch to SSL-encrypted connection)
NEW: Do you have a Google/GoogleMail or Yahoo account? No registration required!
Log in with Google account | Log in with Yahoo account
Log in with Google account | Log in with Yahoo account
No account? Register here.