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/5906#issuecomment-953858365,https://api.github.com/repos/pydata/xarray/issues/5906,953858365,IC_kwDOAMm_X8442rk9,1312546,2021-10-28T13:43:04Z,2021-10-28T13:43:04Z,MEMBER,"There are two changes here 1. Only check the `.data` of non-index variables, done at https://github.com/pydata/xarray/pull/5906/files#diff-763e3002fd954d544b05858d8d138b828b66b6a2a0ae3cd58d2040a652f14638R4161-R4163 2. The check for whether or not a full index was needed is done in a `for dim in dims` loop, but the condition doesn't actually depend on `dim`. So I lifted that check out of the for loop (doesn't matter much, since stuff is cached). cc @dcherian","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,1038531231