New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Delay transforming priority_order into ndarray #1045

Merged

taylorfturner merged 1 commit into capitalone:dev from junholee6a:delay-transform-to-ndarray-2

Oct 27, 2023

+3 −3

Commits on Oct 26, 2023

Delay transforming priority_order into ndarray

In the changed code, we had a mypy error because numpy ndarrays are not
compatible with random.Random.shuffle() (expected argument type is
MutableSequence[Any])

We fix this by first instantiating priority_order as a list, then
shuffling it, then creating an ndarray from it afterwards.

junholee6a committed Oct 26, 2023