You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It doesn't make sense to keep constructors with std::initializer_list for mcg31m1 and mcg59 random generators since one seed value is enough. Size of the state for mcg31m1 is 31 bits, uint32_t is enough to store the initial state. For mcg59 size is 59 bits, uint64_t is enough. So, I'd keep only std::uint32_t seed for mcg31m1 and std::uint64_t seed for mcg59.
It doesn't make sense to keep constructors with
std::initializer_list
formcg31m1
andmcg59
random generators since one seed value is enough. Size of the state formcg31m1
is 31 bits,uint32_t
is enough to store the initial state. Formcg59
size is 59 bits,uint64_t
is enough. So, I'd keep onlystd::uint32_t seed
formcg31m1
andstd::uint64_t seed
formcg59
.PR with bug fixes is here: #525.
Feel free to provide your feedback.
The text was updated successfully, but these errors were encountered: