issue_comments: 907066176

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/pull/5692#issuecomment-907066176 https://api.github.com/repos/pydata/xarray/issues/5692 907066176 IC_kwDOAMm_X842ELtA 4160723 2021-08-27T09:32:57Z 2021-08-27T09:32:57Z MEMBER

With the latest commits, the return type of Index.query() starts to look ugly:

Tuple[Mapping[str, Any], Optional[IndexWithVars]] where IndexWithVars = Tuple["Index", Optional[IndexVars]]

We need to return:

  • positional indexers in the form {"dim_1": [...], "dim_2": [...]}
  • optionally a new Index object
  • optionally new index variable objects
  • perhaps we should also explicitly return the dimensions to rename (e.g., when only one level of the multi-index remains) instead of inferring it later from the returned index variable(s)?

I think anIndexQueryResults class would be better to hold all this information in a cleaner way. What do you think @shoyer?

 
{
    "total_count": 0,
    "+1": 0,
    "-1": 0,
    "laugh": 0,
    "hooray": 0,
    "confused": 0,
    "heart": 0,
    "rocket": 0,
    "eyes": 0
}
   966983801