You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Traceback (most recent call last):
File "<stdin>", line 1, in <module>
File "/home/jpivarski/irishep/awkward/src/awkward/_operators.py", line 75, in func
return ufunc(self, other, out=(self,))
File "/home/jpivarski/irishep/dask-awkward/src/dask_awkward/lib/core.py", line 1625, in __array_ufunc__
return _map_partitions(
File "/home/jpivarski/irishep/dask-awkward/src/dask_awkward/lib/core.py", line 1944, in _map_partitions
meta = map_meta(fn, *args, **kwargs)
TypeError: map_meta() got an unexpected keyword argument 'out'
which is confusing to users. If dask_awkward.Array had
and similar, then one += two would change one in place. I'm told that dask.array and dask-histogram already do this.
The implementation I've suggested above, in which dask-awkward replaces its _meta in __iadd__, is a little different from what I'm planning for Awkward (scikit-hep/awkward#3084), which would replace its layout. Maybe there are bad consequences, like losing typetracer reports. Can a _meta change in place? Maybe it would be better to do this:
def__iadd__(self, other):
self._meta+=other._meta# also propagate other's typetracer reports into self?returnself
Like scikit-hep/awkward#3084, there's been a request to allow
dask_array += something
by replacing the content of thedask_array
. For example,raises
which is confusing to users. If
dask_awkward.Array
hadand similar, then
one += two
would changeone
in place. I'm told thatdask.array
and dask-histogram already do this.The implementation I've suggested above, in which dask-awkward replaces its
_meta
in__iadd__
, is a little different from what I'm planning for Awkward (scikit-hep/awkward#3084), which would replace itslayout
. Maybe there are bad consequences, like losing typetracer reports. Can a_meta
change in place? Maybe it would be better to do this:after scikit-hep/awkward#3084 has been implemented.
The text was updated successfully, but these errors were encountered: