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

Fix random indexer #158

Merged

J535D165 merged 2 commits into J535D165:master from tteigman:fix_random_indexer

Apr 8, 2022

Contributor

tteigman commented Apr 28, 2021

I changed the dtype for the random indexer to np.int64 since it wasn't working with the defaults (for me).

Also, the indexer.random method was missing arguments.

tteigman added 2 commits

April 28, 2021 09:19


          added dtype to prevent valueerror

78dc3d6


          added arguments to random method

86f89ac

J535D165 added the bug label

J535D165 merged commit 81792b7 into J535D165:master

Owner

J535D165 commented Apr 8, 2022

It has been a while, but still appreciated :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug