issue_comments: 392833478

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/1603#issuecomment-392833478 https://api.github.com/repos/pydata/xarray/issues/1603 392833478 MDEyOklzc3VlQ29tbWVudDM5MjgzMzQ3OA== 1217238 2018-05-29T16:04:27Z 2018-05-29T16:04:27Z MEMBER

Sure, this is as good a time as any. But we'll probably need to refinish this refactoring before it makes sense to implement anything.

On Tue, May 29, 2018 at 8:59 AM Alistair Miles notifications@github.com wrote:

Ok, cool. Was wondering if now was right time to revisit that, alongside the work proposed in this PR. Happy to participate in that discussion, still interested in implementing some alternative index classes.

On Tue, 29 May 2018, 15:45 Stephan Hoyer, notifications@github.com wrote:

Yes, the index API still needs to be determined. But I think we want to support something like that. On Tue, May 29, 2018 at 1:20 AM Alistair Miles <notifications@github.com

wrote:

I see this mentions an Index API, is that still to be decided?

On Tue, 29 May 2018, 05:28 Stephan Hoyer, notifications@github.com wrote:

I started thinking about how to do this incrementally, and it occurs to me that a good place to start would be to write some of the utility functions we'll need for this:

  1. Normalizing and creating default indexes in the Dataset/DataArray constructor.
  2. Combining indexes from all xarray objects that are inputs for an operations into indexes for the outputs.
  3. Extracting MultiIndex objects from arguments into Dataset/DataArray and expanding them into multiple variables.

I drafted up docstrings for each of these functions and did a little bit of working starting to think through implementations in #2195 https://github.com/pydata/xarray/pull/2195. So this would be a great place for others to help out. Each of these could be separate PRs.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub <https://github.com/pydata/xarray/issues/1603#issuecomment-392649605 , or mute the thread <

https://github.com/notifications/unsubscribe-auth/AAq8QvMauEPa6hfgorDoShZ2PwyYWk6Tks5t3M6AgaJpZM4PtACU

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pydata/xarray/issues/1603#issuecomment-392692996, or mute the thread <

https://github.com/notifications/unsubscribe-auth/ABKS1p8RjrupPM2z2d4_ylWX7826RQ0Rks5t3QTHgaJpZM4PtACU

.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/pydata/xarray/issues/1603#issuecomment-392803210, or mute the thread < https://github.com/notifications/unsubscribe-auth/AAq8QgygnzTX053NlGZ5A5j_tRkRxMj7ks5t3V79gaJpZM4PtACU

.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/pydata/xarray/issues/1603#issuecomment-392831984, or mute the thread https://github.com/notifications/unsubscribe-auth/ABKS1u3XEy9d3xV4M2LLfshNFWN786h9ks5t3XBzgaJpZM4PtACU . 

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