Skip to content

Fix meta for min/max reductions#6736

Merged
quasiben merged 4 commits intodask:masterfrom
pentschev:fix-min-max-meta
Oct 21, 2020
Merged

Fix meta for min/max reductions#6736
quasiben merged 4 commits intodask:masterfrom
pentschev:fix-min-max-meta

Conversation

@pentschev
Copy link
Member

Fix meta for min/max reductions which prevented output arrays to be of the correct type, e.g., CuPy.

@quasiben
Copy link
Member

Thanks @pentschev . Left one small comment

@pentschev
Copy link
Member Author

Anything else here @quasiben ?

@quasiben
Copy link
Member

Nope. I'll plan to merge in a few hours unless other maintainers have thoughts here

@quasiben
Copy link
Member

Thanks again @pentschev . Merging in

@quasiben quasiben merged commit 0e406d9 into dask:master Oct 21, 2020
kumarprabhu1988 pushed a commit to kumarprabhu1988/dask that referenced this pull request Oct 29, 2020
* Fix meta for min/max reductions

* Add more CuPy reduction tests

* Fix compute_meta ValueError exception handling
gjoseph92 added a commit to gjoseph92/dask that referenced this pull request May 18, 2021
`args_meta[0]` could be wrong when there are multiple arguments. Even this I don't really like, since it's making a lot of assumptions about what `func` might be that may not be true in generality for any user-defined function.
gjoseph92 added a commit to gjoseph92/dask that referenced this pull request May 18, 2021
`args_meta[0]` could be wrong when there are multiple arguments. Even this I don't really like, since it's making a lot of assumptions about what `func` might be that may not be true in generality for any user-defined function.
gjoseph92 added a commit to gjoseph92/dask that referenced this pull request May 18, 2021
`args_meta[0]` could be wrong when there are multiple arguments. Even this I don't really like, since it's making a lot of assumptions about what `func` might be that may not be true in generality for any user-defined function.
jsignell pushed a commit that referenced this pull request Jun 3, 2021
* Array: correct number of outputs in `apply_gufunc`

* Stricter #6736

`args_meta[0]` could be wrong when there are multiple arguments. Even this I don't really like, since it's making a lot of assumptions about what `func` might be that may not be true in generality for any user-defined function.

* keep std
@pentschev pentschev deleted the fix-min-max-meta branch June 30, 2021 12:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants