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

Add scikit-learn to “test” extra #48

Merged

TomGeorge1234 merged 1 commit into RatInABox-Lab:dev from musicinmybrain:sklearn

May 23, 2023

Contributor

musicinmybrain commented May 23, 2023

It is used in tests/test_advanced.py:

https://github.com/TomGeorge1234/RatInABox/blob/b8d9103bb77bbb42ddeb1abdfcfa2b3c126d43b7/tests/test_advanced.py#L5

https://github.com/TomGeorge1234/RatInABox/blob/b8d9103bb77bbb42ddeb1abdfcfa2b3c126d43b7/tests/test_advanced.py#L13-L14


          Add scikit-learn to “test” extra

0dd1ae2

It is used in tests/test_advanced.py

TomGeorge1234 approved these changes

View reviewed changes

Collaborator

TomGeorge1234 left a comment

must be specific to some machines I've never run into this but seems good to me, thanks for the spot.

TomGeorge1234 merged commit 4629968 into RatInABox-Lab:dev

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

Labels

None yet