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/3498#issuecomment-573325829,https://api.github.com/repos/pydata/xarray/issues/3498,573325829,MDEyOklzc3VlQ29tbWVudDU3MzMyNTgyOQ==,35968931,2020-01-11T15:21:51Z,2020-01-11T15:21:51Z,MEMBER,"I think there's nothing left to do here, thanks @juseg!","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,520110180 https://github.com/pydata/xarray/pull/3498#issuecomment-565725336,https://api.github.com/repos/pydata/xarray/issues/3498,565725336,MDEyOklzc3VlQ29tbWVudDU2NTcyNTMzNg==,35968931,2019-12-14T15:04:09Z,2019-12-14T15:04:09Z,MEMBER,"Thanks @juseg . > in Python 3.6- the order is not guaranteed preserved. I think for python 3.6 and above the [order is preserved](https://stackoverflow.com/questions/39980323/are-dictionaries-ordered-in-python-3-6) isn't it? > the current default 0 is not well defined in case of nested lists. Yes, this is what I was thinking of. > the ids are essentially ND indexes that could perhaps be used... We *could* do this, and that's how we would solve it in general, but I don't really think it's worth the effort/complexity. > Or should we just stick to file paths as you suggest? And leave the default as is I think so - if we do this then users can still easily pick the attributes from the file of their choosing (solving the original issue), and if someone wants to be able to choose the `attrs_file` in another way later then we can worry about that then.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,520110180 https://github.com/pydata/xarray/pull/3498#issuecomment-565622102,https://api.github.com/repos/pydata/xarray/issues/3498,565622102,MDEyOklzc3VlQ29tbWVudDU2NTYyMjEwMg==,35968931,2019-12-13T21:53:08Z,2019-12-13T21:53:08Z,MEMBER,"I'm not sure we should merge changes if we're unsure how they will behave in certain circumstances. > On the other hand I am eager to keep the file number option because (1) attrs_file=-1 is the behaviour that I need If we kept just the string specifier, you could still solve the problem of preserving the history: ```python files_to_open = ['filepath1', 'filepath2'] ds = open_mfdataset(files_to_open, attrs_file=files_to_open[-1]) ``` But then the option would always have clear and well-defined behaviour, even in more complex cases like `combine_by_coords`or `combine='nested'` with a >1D input file list. ","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,520110180 https://github.com/pydata/xarray/pull/3498#issuecomment-565413630,https://api.github.com/repos/pydata/xarray/issues/3498,565413630,MDEyOklzc3VlQ29tbWVudDU2NTQxMzYzMA==,35968931,2019-12-13T11:48:58Z,2019-12-13T21:48:24Z,MEMBER,"Thanks for this @juseg. The only problem I see is that a scalar number to specify the file only makes sense if it's a 1D list, but `open_mfdataset` can also accept nested list-of-lists (with `combine='nested'`), or ignore the order of the input entirely (with `combine='by_coords'`). What happens if you pass a list-of-lists of datasets? On the other hand specifying the particular filepath or object makes sense in all cases, so perhaps the easiest way to avoid ambiguity would be to restrict to that option? (The default would just be left as-is.)","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,520110180