Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: Peter Andreas Entschev <[email protected]>
  • Loading branch information
rjzamora and pentschev authored Sep 10, 2024
1 parent 89027be commit 262e52f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion docs/source/examples/best-practices.rst
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ Additionally, when using `Accelerated Networking`_ , we only need to register a
Spilling from Device
~~~~~~~~~~~~~~~~~~~~

Dask CUDA offers several different ways to enable automatic spilling from device memory.
Dask-CUDA offers several different ways to enable automatic spilling from device memory.
The best method often depends on the specific workflow. For classic ETL workloads with
`Dask cuDF <https://docs.rapids.ai/api/dask-cudf/stable/>`_, cuDF spilling is usually
the best place to start. See `spilling`_ for more details.
Expand Down
2 changes: 1 addition & 1 deletion docs/source/spilling.rst
Original file line number Diff line number Diff line change
Expand Up @@ -169,4 +169,4 @@ Although cuDF spilling is the best option for most Dask cuDF ETL workflows,
it will be much less effective if that workflow converts between ``cudf.DataFrame``
and other data formats (e.g. ``cupy.ndarray``). Once the underlying device buffers
are "exposed" to external memory references, they become "unspillable" by cuDF.
In cases like this (e.g. Dask CUDA + XGBoost), JIT-Unspill is usually a better choice.
In cases like this (e.g., Dask-CUDA + XGBoost), JIT-Unspill is usually a better choice.

0 comments on commit 262e52f

Please sign in to comment.