Skip to content
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.

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

Remove `threads` arg of `install_deps()` #154

Merged
merged 1 commit into from Sep 10, 2018

Conversation

Projects
None yet
2 participants
@gaborcsardi
Copy link
Member

commented Sep 10, 2018

If Ncpus was also specified, that led to an error.
Closes #153.

@gaborcsardi gaborcsardi force-pushed the fix/ncpu branch 2 times, most recently from f23b37b to 85e1601 Sep 10, 2018

@jimhester

This comment has been minimized.

Copy link
Member

commented Sep 10, 2018

LGTM

@jimhester jimhester self-requested a review Sep 10, 2018

Remove `threads` arg of `install_deps()`
If `Ncpus` was also specified, that led to an error.
Closes #153.

@gaborcsardi gaborcsardi force-pushed the fix/ncpu branch from 85e1601 to 25ca7e1 Sep 10, 2018

@gaborcsardi gaborcsardi merged commit aaa1fd3 into master Sep 10, 2018

0 of 2 checks passed

continuous-integration/appveyor/branch Waiting for AppVeyor build to complete
Details
continuous-integration/appveyor/pr Waiting for AppVeyor build to complete
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.