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
I've seen you are using math/rand, which causes diversity and gender quota issues ;-)
I often ended up with the same gender
I often had collisions, with the same names being chosen in a row
I suggest changing random number generation from
import "math/rand"
// ...
g := rand.Intn(2)
to
import "crypto/rand"
// ...
n, _ := rand.Int(rand.Reader, big.NewInt(2))
g := int(n.Int64())
(Respectively, something similar in func randomFrom(...))
I could mitigate the gender selection by generating the random number myself, but the name for that gender is then still decided by internals and not properly random, often causing the same name in a row.
The text was updated successfully, but these errors were encountered:
Hi,
I've seen you are using math/rand, which causes diversity and gender quota issues ;-)
I suggest changing random number generation from
to
(Respectively, something similar in
func randomFrom(...)
)I could mitigate the gender selection by generating the random number myself, but the name for that gender is then still decided by internals and not properly random, often causing the same name in a row.
The text was updated successfully, but these errors were encountered: