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 default take value of train and predict methods #48

Merged

yonromai merged 1 commit into main from romain/fix-take-default

Oct 13, 2023

Contributor

yonromai commented Oct 13, 2023

Bug: By default all nodes should be considered (take=None) rather than 0.

(Related to #35)


          Fix default take value of train and predict methods

fffeaf8

yonromai requested a review from dhimmel

October 13, 2023 18:07

Contributor Author

yonromai commented Oct 13, 2023

@dhimmel Merging right away. Please let me know in #35 if you run into any other issue. Thanks for your help!

yonromai merged commit 3c6cc0f into main

2 checks passed

yonromai deleted the romain/fix-take-default branch

October 13, 2023 18:10

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

Labels

None yet