Skip to content

Comments

Fixing issue https://github.com/dask/dask-examples/issues/235#245

Open
srajesh2712 wants to merge 2 commits intodask:mainfrom
srajesh2712:update-XGboost-example
Open

Fixing issue https://github.com/dask/dask-examples/issues/235#245
srajesh2712 wants to merge 2 commits intodask:mainfrom
srajesh2712:update-XGboost-example

Conversation

@srajesh2712
Copy link
Contributor

Fixed the mode import as it was deprecated
Fixed the training and prediction logic

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@srajesh2712
Copy link
Contributor Author

@jacobtomlinson could you take a look at this whenever possible.

@jacobtomlinson
Copy link
Member

I've merged a couple of text tweaks recently, but I'm not comfortable reviewing/merging code changes until we get CI unblocked #246

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants