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

[Doc] Expose train_test_split in API doc #4234

Merged
merged 1 commit into from
Sep 25, 2021

Conversation

hcho3
Copy link
Contributor

@hcho3 hcho3 commented Sep 24, 2021

Currently train_test_split isn't showing up in the cuML API doc.

@hcho3 hcho3 added doc Documentation non-breaking Non-breaking change labels Sep 24, 2021
@dantegd dantegd added this to PR-WIP in v21.10 Release via automation Sep 24, 2021
@dantegd
Copy link
Member

dantegd commented Sep 24, 2021

@gpucibot merge

@hcho3
Copy link
Contributor Author

hcho3 commented Sep 25, 2021

rerun tests

@rapids-bot rapids-bot bot merged commit b0b2e79 into rapidsai:branch-21.10 Sep 25, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #4234 (a34c79a) into branch-21.10 (a69db68) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff                @@
##           branch-21.10    #4234      +/-   ##
================================================
+ Coverage         86.07%   86.09%   +0.01%     
================================================
  Files               231      231              
  Lines             18649    18687      +38     
================================================
+ Hits              16052    16088      +36     
- Misses             2597     2599       +2     
Flag Coverage Δ
dask 47.12% <ø> (+0.09%) ⬆️
non-dask 78.77% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
python/cuml/linear_model/linear_regression.pyx 92.68% <0.00%> (+0.91%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a69db68...a34c79a. Read the comment docs.

v21.10 Release automation moved this from PR-WIP to Done Sep 25, 2021
vimarsh6739 pushed a commit to vimarsh6739/cuml that referenced this pull request Oct 9, 2023
Currently `train_test_split` isn't showing up in the cuML API doc.

Authors:
  - Philip Hyunsu Cho (https://github.com/hcho3)

Approvers:
  - Dante Gama Dessavre (https://github.com/dantegd)

URL: rapidsai#4234
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Documentation non-breaking Non-breaking change
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants