issue_comments: 535010456
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/3315#issuecomment-535010456 | https://api.github.com/repos/pydata/xarray/issues/3315 | 535010456 | MDEyOklzc3VlQ29tbWVudDUzNTAxMDQ1Ng== | 35968931 | 2019-09-25T13:00:13Z | 2019-09-25T13:00:13Z | MEMBER | Okay something has definitely gone wrong here. My intention with that test was to check that the order of operations doesn't matter, but you're right that the test as written makes no sense. It would probably be a good idea to remove this test and check that property correctly by adding a second assert to the (poorly-named) Prove it works symmetricallydatasets = [[ds(0), ds(3)], [ds(1), ds(4)], [ds(2), ds(5)]] result = combine_nested(datasets, concat_dim=["dim2", "dim1"]) assert_equal(result, expected) ``` (This passes fine) However, that still leaves the question of why is this nonsensical test passing? I think it's because result = concat([da1, da2], dim="x")
|
{ "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
494906646 |