Project

General

Profile

Actions

Task #1369

closed

Unify randomization using Boost.Random library

Added by Alex Afanasyev about 10 years ago. Updated almost 10 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.

Actions #1

Updated by Junxiao Shi almost 10 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
Actions #2

Updated by Alex Afanasyev almost 10 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.

Actions #3

Updated by Junxiao Shi almost 10 years ago

  • Category set to Core
Actions #4

Updated by Junxiao Shi almost 10 years ago

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

20140530 conference call approves the approach in note-1.

Actions #5

Updated by Junxiao Shi almost 10 years ago

  • Status changed from New to In Progress
Actions #6

Updated by Junxiao Shi almost 10 years ago

  • % Done changed from 0 to 70
Actions #7

Updated by Junxiao Shi almost 10 years ago

  • Status changed from In Progress to Code review
Actions #8

Updated by Junxiao Shi almost 10 years ago

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

Also available in: Atom PDF