Skip to content

Expand dask.utils.is_arraylike docstring.#7445

Merged
jrbourbeau merged 3 commits intodask:mainfrom
douglasdavis:docstring-is_arraylike
Mar 22, 2021
Merged

Expand dask.utils.is_arraylike docstring.#7445
jrbourbeau merged 3 commits intodask:mainfrom
douglasdavis:docstring-is_arraylike

Conversation

@douglasdavis
Copy link
Member

Specifically to point out difference between what the function is checking and how NumPy defines array_like

@jakirkham
Copy link
Member

cc @shoyer @pentschev @jrbourbeau @jsignell @mrocklin (in case any of you have thoughts/suggestions on this change)

Copy link
Member

@shoyer shoyer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, thanks!

Copy link
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @douglasdavis! I left a couple of small, non-blocking comments

@douglasdavis
Copy link
Member Author

thanks @jrbourbeau changes incoprorated 👍

Copy link
Member

@pentschev pentschev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, suggested a minor typo fix, thanks @douglasdavis !

Co-authored-by: Peter Andreas Entschev <peter@entschev.com>
Copy link
Member

@jrbourbeau jrbourbeau left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @douglasdavis! This is in

@jrbourbeau jrbourbeau merged commit f6e9e81 into dask:main Mar 22, 2021
@douglasdavis douglasdavis deleted the docstring-is_arraylike branch March 23, 2021 02:00
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.

Rename is_arraylike to is_duck_array

5 participants