pull_requests: 314874029
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 314874029 | MDExOlB1bGxSZXF1ZXN0MzE0ODc0MDI5 | 3283 | open | 0 | Add hypothesis test for netCDF4 roundtrip | 327925 | Part of #1846: add a property-based test for reading & writing netCDF4 files. This is the first time I've played with Hypothesis, but it seems to be working - e.g. I got an error with float16, and the [netCDF docs show](https://www.unidata.ucar.edu/software/netcdf/docs/data_type.html) that 16-bit floats are not a supported data type. However: - This currently only tests a dataset with a single variable - it could be extended to multiple variables if that's useful. - It [looks like](https://www.unidata.ucar.edu/software/netcdf/docs/netcdf_data_set_components.html#Permitted) netCDF4 should support unicode characters, but it failed when I didn't have `max_codepoint=255` in there. I don't know if that's an expected limitation I'm not aware of, or a bug somewhere. But I thought I'd make the test pass for now. | 2019-09-06T09:33:48Z | 2022-11-21T22:45:13Z | e1c54ec5ff1e8fe66a9e3f55a805414fe11b6480 | 0 | fbeaf41971afe2b74378d7de4807575d033bdb24 | d1e4164f3961d7bbb3eb79037e96cae14f7182f8 | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/3283 |
Links from other tables
- 2 rows from pull_requests_id in labels_pull_requests