issue_comments: 524518305
This data as json
html_url | issue_url | id | node_id | user | created_at | updated_at | author_association | body | reactions | performed_via_github_app | issue |
---|---|---|---|---|---|---|---|---|---|---|---|
https://github.com/pydata/xarray/issues/525#issuecomment-524518305 | https://api.github.com/repos/pydata/xarray/issues/525 | 524518305 | MDEyOklzc3VlQ29tbWVudDUyNDUxODMwNQ== | 3460034 | 2019-08-24T04:17:54Z | 2019-08-24T04:17:54Z | CONTRIBUTOR | With the progress being made with https://github.com/pydata/xarray/pull/2956, https://github.com/pydata/xarray/pull/3238, and https://github.com/hgrecco/pint/pull/764, I was thinking that now might be a good time to work out the details of the "minimal units layer" mentioned by @shoyer in https://github.com/pydata/xarray/issues/525#issuecomment-482641808 and https://github.com/pydata/xarray/issues/988#issuecomment-413732471? I'd be glad to try putting together a PR that could follow up on https://github.com/pydata/xarray/pull/3238 for it, but I would want to ask for some guidance: (For reference, below is the action list from https://github.com/pydata/xarray/issues/988#issuecomment-413732471)
Having
Units and IO While wrapping and unwrapping arrays with Some possibilities that came to mind (by no means an exhaustive list):
With any of these, tests for lazy-loading would be crucial (I don't know yet how pint will handle that). Output may be easier: I was thinking that unwrapping could be done implicitly by automatically putting Extra questions based on sparse implementation Will a set of repr functions for each unit array type need to be added like they were for sparse in https://github.com/pydata/xarray/pull/3211? Or should there be some more general system implemented because of all of the possible combinations that would arise with other duck array types?
What is the expected behavior with unit arrays with regards to this soon-to-be-implemented conversion method? |
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
100295585 |