all: consider using math/rand/v2 when available #1906
Comments
|
I was thinking the same thing. |
|
Go 1.23 is released https://go.dev/blog/go1.23 We can start working on this after we bump our minimum supported version to Go 1.22 1. Footnotes |
|
I have already started work on this (against 1.23r1). The changes are not trivial. |
|
The complexities arise from differences in sequencing between exp/rand and rand/v2. There are also API differences which complicate things. I forked out a copy of rand/v2 into an internal package to shim over the API differences with the aim to progressively repair the change. Due to the sequence differences, some tests will no longer pass (catastrophically in some cases). I may send the branch as I have it at the moment, but I don't have time to work on it for the next few weeks. |
|
Change as it exists, is here. |
as golang/go#61716 is now accepted, we should probably migrate to using
math/rand/v2, when all the Go versions we support ship it.(and drop the use of
x/exp/rand)The text was updated successfully, but these errors were encountered: