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/3032#issuecomment-503746105,https://api.github.com/repos/pydata/xarray/issues/3032,503746105,MDEyOklzc3VlQ29tbWVudDUwMzc0NjEwNQ==,1217238,2019-06-19T21:04:56Z,2019-06-19T21:04:56Z,MEMBER,"With NumPy arrays at least, there is no cost for broadcasting, because it can always be done with views. But even for other array types, inserting size 1 dimensions in the correct location should be basically free, and would be more helpful than what we currently do On Wed, Jun 19, 2019 at 9:25 PM Oriol Abril wrote: > I'm trying to think whether there would be any performance cost there - > i.e. are there any arrays where preemptive broadcasting would be both > expensive and unnecessary? > > Even if there were a performance cost (compared to the actual behaviour), > it could be easily avoided by using all dims as input_core_dims couldn't > it? IIUC, all dims should be broadcasted unless they are in input core > dims, so it broadcasting could still be avoided without problem. > > — > You are receiving this because you commented. > Reply to this email directly, view it on GitHub > , > or mute the thread > > . > ","{""total_count"": 2, ""+1"": 2, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,457716471 https://github.com/pydata/xarray/issues/3032#issuecomment-503650578,https://api.github.com/repos/pydata/xarray/issues/3032,503650578,MDEyOklzc3VlQ29tbWVudDUwMzY1MDU3OA==,1217238,2019-06-19T17:15:32Z,2019-06-19T17:15:32Z,MEMBER,"Yes, exactly.","{""total_count"": 3, ""+1"": 3, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,457716471 https://github.com/pydata/xarray/issues/3032#issuecomment-503618955,https://api.github.com/repos/pydata/xarray/issues/3032,503618955,MDEyOklzc3VlQ29tbWVudDUwMzYxODk1NQ==,1217238,2019-06-19T15:50:03Z,2019-06-19T15:50:03Z,MEMBER,"For what it's worth, I agree that this behavior is a little surprising. We should probably make `apply_ufunc()` explicitly broadcast arrays first.","{""total_count"": 0, ""+1"": 0, ""-1"": 0, ""laugh"": 0, ""hooray"": 0, ""confused"": 0, ""heart"": 0, ""rocket"": 0, ""eyes"": 0}",,457716471