Project

General

Profile

Task #1369

Unify randomization using Boost.Random library

Added by Alex Afanasyev over 5 years ago. Updated over 5 years ago.

Status:
Closed
Priority:
Low
Assignee:
Category:
Core
Target version:
Start date:
Due date:
% Done:

100%

Estimated time:
3.00 h

Description

Currently we have several places where random variables are used in a different way with different approaches to seeding. CS uses srand and rand, ncc strategy uses nrand48. Since we now have ability to use Boost.Random library, the use of random variables should be abstracted and unified.

History

#1 Updated by Junxiao Shi over 5 years ago

I think it's unnecessary to abstract randomization.

We just need to unify randomization:

  • provide a global Uniform Random Number Generator instance, by default mt19937_64
  • let each module have its own Random Distribution

#2 Updated by Alex Afanasyev over 5 years ago

This wasn't about abstract randomization, rather about the fact that we have 4 different mechanisms to get random numbers: with boost random, with nrand, with just rand, and with cryptopp random generator. There shouldn't be such variety, since it almost impossible to properly seed the random number generator when the need arises.

#3 Updated by Junxiao Shi over 5 years ago

  • Category set to Core

#4 Updated by Junxiao Shi over 5 years ago

  • Assignee set to Junxiao Shi
  • Estimated time set to 3.00 h

20140530 conference call approves the approach in note-1.

#5 Updated by Junxiao Shi over 5 years ago

  • Status changed from New to In Progress

#6 Updated by Junxiao Shi over 5 years ago

  • % Done changed from 0 to 70

#7 Updated by Junxiao Shi over 5 years ago

  • Status changed from In Progress to Code review

#8 Updated by Junxiao Shi over 5 years ago

  • Status changed from Code review to Closed
  • % Done changed from 70 to 100

Also available in: Atom PDF