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/2492#issuecomment-431559524,https://api.github.com/repos/pydata/xarray/issues/2492,431559524,MDEyOklzc3VlQ29tbWVudDQzMTU1OTUyNA==,226037,2018-10-20T08:11:03Z,2018-10-20T08:11:03Z,MEMBER,"@shoyer I consider https://github.com/pydata/xarray/pull/2492/commits/486a391915d3bf1af685ae9ef908c30d5e053851 in a mergiable state if you like it.
I updated the description of the PR to sync with the new aim.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,371514413
https://github.com/pydata/xarray/pull/2492#issuecomment-431477498,https://api.github.com/repos/pydata/xarray/issues/2492,431477498,MDEyOklzc3VlQ29tbWVudDQzMTQ3NzQ5OA==,226037,2018-10-19T19:44:33Z,2018-10-19T19:45:48Z,MEMBER,"And we ""refuse to guess""!
I broke the forrest of `try / except` to different functions with no real change (hopefully!) except `_get_default_engine_grib`.
Now opening a GRIB file without explicit `engine` raises an exception, but the error message suggests to the user to use the installed backend or to install one if none is present.
@shoyer if this too complex? Alternatively I can always return the same, generic, error message.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,371514413
https://github.com/pydata/xarray/pull/2492#issuecomment-431297953,https://api.github.com/repos/pydata/xarray/issues/2492,431297953,MDEyOklzc3VlQ29tbWVudDQzMTI5Nzk1Mw==,226037,2018-10-19T09:09:54Z,2018-10-19T09:38:30Z,MEMBER,"@shoyer you are right, the GRIB translation is not unique and with *cfgrib* we have no plan to keep compatibility with *PyNIO*. I also agree that this will be really poor user experience πin some circumstances (even if in general it will work just fine). **Do not merge yet!**
We may default to one of the two without fall-back if it is not present (you may guess that I have a bias on which one to choose π), or keep the current behaviour.
In any event we can improve the error message by suggesting to add the `engine=` corresponding to the installed backends.
I'll show you what I mean.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,371514413
https://github.com/pydata/xarray/pull/2492#issuecomment-431082419,https://api.github.com/repos/pydata/xarray/issues/2492,431082419,MDEyOklzc3VlQ29tbWVudDQzMTA4MjQxOQ==,226037,2018-10-18T16:50:00Z,2018-10-18T16:50:20Z,MEMBER,"Mm... the private functions in `api.py` are not unit tested as they are not part of the API, I'll add a `test_backends_api.py` for the `_get_default_engine` tests and then others can use it to unit test the other `_` functions. ","{""total_count"": 1, ""+1"": 1, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,371514413