pull_requests: 447356800
This data as json
id | node_id | number | state | locked | title | user | body | created_at | updated_at | closed_at | merged_at | merge_commit_sha | assignee | milestone | draft | head | base | author_association | auto_merge | repo | url | merged_by |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
447356800 | MDExOlB1bGxSZXF1ZXN0NDQ3MzU2ODAw | 4214 | closed | 0 | Add initial cupy tests | 1610850 | Added some initial unit tests for cupy. Mainly to create a place for cupy tests to go and to check some basic functionality. I've created a fixture which constructs the dataset from the Toy weather data example and converts the underlying arrays to cupy. Then I've added a test which checks that after calling operations such as `mean` and `groupby` the resulting DataArray is still backed by a cupy array. The main penalty with working on GPUs is accidentally shunting data back and forth between the GPU and system memory. Copying data over the PCI bus is slow compared to the rest of the work so should be avoided. So this first test is checking that we are leaving things on the GPU. Because this data copying is so expensive cupy have intentionally broken the `__array__` method and introduced a `.get` method instead. This means that users have to be explicit in converting back to numpy and copying back to the main memory. Therefore we will need to add some logic to xarray to use `.get` in appropriate situations such as plotting. - [x] Releated to #4212 - [x] Tests added - [x] Passes `isort -rc . && black . && mypy . && flake8` | 2020-07-10T10:20:33Z | 2020-07-13T16:32:35Z | 2020-07-13T15:07:45Z | 2020-07-13T15:07:44Z | 52043bc57f20438e8923790bca90b646c82442ad | 0 | cff60a17ee7a5e05033711578e28ed94acb73121 | 7bf9df9d75c40bcbf2dd28c47204529a76561a3f | CONTRIBUTOR | 13221727 | https://github.com/pydata/xarray/pull/4214 |
Links from other tables
- 1 row from pull_requests_id in labels_pull_requests