Skip to content

Commit

Permalink
Make warning against GPU allocations more prominent.
Browse files Browse the repository at this point in the history
  • Loading branch information
vyasr committed Nov 2, 2022
1 parent e18f5c5 commit 0d622b5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion docs/cudf/source/developer_guide/testing.md
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ For the purpose of this discussion,
we define a parametrization as "simple" if it is composed of a list (possibly nested) of primitive objects.
Examples include a list of integers or a list of list of strings.
This _does not_ include e.g. cuDF or pandas objects.
In particular, developers should avoid performing GPU memory allocations during test collection.

With that in mind, here are some ground rules for how to parametrize.

Expand Down Expand Up @@ -138,7 +139,6 @@ def test_odds():
```

Other approaches are also possible, and the best solution should be discussed on a case-by-case basis during PR review.
Developers should avoid performing GPU memory allocations during test collection.

### Testing utility functions

Expand Down

0 comments on commit 0d622b5

Please sign in to comment.