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