define fill value for where

[mathause]

It would be nice if where accepts an other argument:

def where(self, cond, other=np.NaN):
    pass

[shoyer]

Agreed. I didn't add this only because we don't yet have three-argument broadcasting operations working yet.

For now, a work around (at a slight cost in efficiency) is to use where followed by fillna.

[max-sixty]

This would be great to add - we just hit this (CC @justinkuosixty).

For now, a work around (at a slight cost in efficiency) is to use where followed by fillna.

This works, although if there are already NaNs, it requires another where to add those back

[jhamman]

@shoyer - what do we need in order to get the 3-argument broadcasting working. I'd really like to get this into xarray.

[shoyer]

It should already work with apply_ufunc -- we just need to adjust where to use it. Take a look at the latest version of fillna for an example.

…

On Tue, Jan 31, 2017 at 12:35 PM Joe Hamman ***@***.***> wrote: @shoyer <https://github.com/shoyer> - what do we need in order to get the 3-argument broadcasting working. I'd really like to get this into xarray. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#576 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ABKS1l3x4bNqrxkH0X1Ag4kI-T1Yq_Lrks5rX5sUgaJpZM4F9tqJ> .