issues: 314239017
This data as json
id | node_id | number | title | user | state | locked | assignee | milestone | comments | created_at | updated_at | closed_at | author_association | active_lock_reason | draft | pull_request | body | reactions | performed_via_github_app | state_reason | repo | type |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
314239017 | MDU6SXNzdWUzMTQyMzkwMTc= | 2055 | Documentation on assign a value and vectorized indexing | 8453445 | closed | 0 | 10 | 2018-04-13T20:22:18Z | 2018-05-16T02:13:26Z | 2018-05-16T02:13:26Z | CONTRIBUTOR | I was trying to assign a value to a dataset and kept getting no error but also not getting what I wanted. I then was directed to the Warning at the end of the Assigning values with indexing and I realized I wasted a lot of time on something that is not possible. So I am suggesting a few improvements (some might be feasible, some might not): A) I am not sure if it is possible, but maybe add a proper error to such thing - try to assign values when using any of the indexing methods - would be great.
B) if A is not possible, maybe in the DataArray page you should repeat the Warning. In this second page, in fact, you state: "Select or assign values by integer location (like numpy) : x[:10] or by label (like pandas): x.loc['2014-01-01'] or x.sel(time='2014-01-01')." Which I think it's in contradiction, or at least it's not crystal clear.
C) you should add to the text of the Warning to use vectorized indexing, so people know how to fix the issue. ind_y = da.y=='c' da[ind_x, ind_y] =30 ``` E) If I am completely off in the solution that i used in the code above, then add an example that takes care of this. In your examples you use 0s and 1s, which is not what you want to do if you have multiple lat and lon and time coordinates that you want to use correctly. I hope I made some sense.. |
{ "url": "https://api.github.com/repos/pydata/xarray/issues/2055/reactions", "total_count": 0, "+1": 0, "-1": 0, "laugh": 0, "hooray": 0, "confused": 0, "heart": 0, "rocket": 0, "eyes": 0 } |
completed | 13221727 | issue |