Skip to content

Clarify overflow behavior in asarray #409

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 3 commits into from
Apr 7, 2022
Merged

Clarify overflow behavior in asarray #409

merged 3 commits into from
Apr 7, 2022

Conversation

kgryte
Copy link
Contributor

@kgryte kgryte commented Mar 24, 2022

This PR

  • adds a note to asarray concerning what happens when an input value exceeds the precision of the resolved output array data type. Namely, behavior is left unspecified and, thus, implementation-defined. This follows similar guidance as found in full.
  • clarifies in full and full_like that overflow behavior is implementation-defined even when dtype is not None.

@kgryte kgryte added Narrative Content Narrative documentation content. topic: Creation Array creation. labels Mar 24, 2022
@kgryte kgryte requested a review from honno March 24, 2022 09:31
Copy link
Member

@honno honno left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@rgommers rgommers left a comment

Choose a reason for hiding this comment

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

Makes sense, thanks @kgryte. I'll leave it open for a while to give people time to comment.

@kgryte
Copy link
Contributor Author

kgryte commented Apr 7, 2022

As this PR has received 3 approvals, will merge. Any further updates can be addressed in follow-up PRs...

@kgryte kgryte merged commit c580517 into main Apr 7, 2022
@kgryte kgryte deleted the asarray-note branch April 7, 2022 03:37
@kgryte kgryte added this to the v2021 milestone Dec 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Narrative Content Narrative documentation content. topic: Creation Array creation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants