pull_requests: 356084487
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 |
|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
| 356084487 | MDExOlB1bGxSZXF1ZXN0MzU2MDg0NDg3 | 3652 | closed | 0 | Use encoding['dtype'] over data.dtype when possible within CFMaskCoder.encode | 6628425 | <!-- Feel free to remove check-list items aren't relevant to your change --> This uses `encoding['dtype']` over `data.dtype` when possible within `CFMaskCoder.encode` to decide what type to cast `encoding['missing_value']` or `encoding['_FillValue']` to; this is one way to fix #3624. Another possible way would be to ensure the times have the proper `dtype` coming from `CFDatetimeCoder.encode`. I'm not sure what is the preferred solution. cc: @andersy005, @spencerahill - [x] Closes #3624 - [x] Tests added - [x] Passes `black . && mypy . && flake8` - [x] Fully documented, including `whats-new.rst` for all changes and `api.rst` for new API | 2019-12-22T13:05:18Z | 2020-01-15T15:23:41Z | 2020-01-15T15:22:30Z | 2020-01-15T15:22:30Z | 99594051ef591f12b4b78a8b24136da46d0bf28f | 0 | a46efa0787b74475b2fb0a4fb445cf0c327b705c | e0fd48052dbda34ee35d2491e4fe856495c9621b | MEMBER | 13221727 | https://github.com/pydata/xarray/pull/3652 |
Links from other tables
- 0 rows from pull_requests_id in labels_pull_requests