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

[WIP] Remove old dask-cudf documentation and add README #6665

Closed
wants to merge 3 commits into from

Conversation

rjzamora
Copy link
Member

@rjzamora rjzamora commented Nov 4, 2020

Closes #6583

TODO:

  • Check that deleted content is covered elsewhere (otherwise the page should be replaced, rather than deleted).
  • Clean up simple README and make sure proper links are included.

@GPUtester
Copy link
Collaborator

Please update the changelog in order to start CI tests.

View the gpuCI docs here.

@codecov
Copy link

codecov bot commented Nov 4, 2020

Codecov Report

Merging #6665 into branch-0.17 will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@               Coverage Diff               @@
##           branch-0.17    #6665      +/-   ##
===============================================
- Coverage        82.33%   82.31%   -0.03%     
===============================================
  Files               94       94              
  Lines            15369    15366       -3     
===============================================
- Hits             12654    12648       -6     
- Misses            2715     2718       +3     
Impacted Files Coverage Δ
python/cudf/cudf/utils/gpu_utils.py 53.65% <0.00%> (-4.88%) ⬇️
python/cudf/cudf/core/abc.py 87.23% <0.00%> (-4.26%) ⬇️
python/cudf/cudf/_fuzz_testing/fuzzer.py 0.00% <0.00%> (ø)
python/cudf/cudf/utils/hash_vocab_utils.py 100.00% <0.00%> (ø)

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 21a0a33...b8ac7de. Read the comment docs.

@kkraus14
Copy link
Collaborator

kkraus14 commented Feb 2, 2021

@rjzamora should we close this PR?

@kkraus14 kkraus14 changed the base branch from branch-0.17 to branch-0.19 February 2, 2021 21:33
@kkraus14 kkraus14 added dask Dask issue doc Documentation non-breaking Non-breaking change labels Feb 2, 2021
@rjzamora
Copy link
Member Author

rjzamora commented Feb 2, 2021

@rjzamora should we close this PR?

Ah, sorry. This fell off my radar. It is probably fine to close this and replace it with something fresh for 0.19. I wouldn't consider this a huge priority.

@kkraus14 kkraus14 closed this Feb 3, 2021
rapids-bot bot pushed a commit that referenced this pull request Nov 10, 2022
This PR removes an outdated notebook for "Multi-GPU with Dask-cuDF" from the docs. Resolves #6583 with some of the changes from #6665.

See also: rapidsai/rapids.ai#256 (comment)

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - GALI PREM SAGAR (https://github.com/galipremsagar)
  - Richard (Rick) Zamora (https://github.com/rjzamora)

URL: #12095
@rjzamora rjzamora deleted the dask-cudf-readme branch November 10, 2022 16:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dask Dask issue doc Documentation non-breaking Non-breaking change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[DOC] Remove outdated multi-gpu dask-cudf overview from docs
3 participants