-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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
DOC: Improved documentation for numpy.frombuffer #23556
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! A few comments below
364cf60
to
7aa931d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks!
jax/_src/numpy/lax_numpy.py
Outdated
def frombuffer(buffer: bytes | Any, dtype: DTypeLike = float, | ||
count: int = -1, offset: int = 0) -> Array: | ||
"""Convert a buffer into a 1-D JAX array. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should fix the test failure:
"""Convert a buffer into a 1-D JAX array. | |
r"""Convert a buffer into a 1-D JAX array. |
7aa931d
to
b2ab9d8
Compare
Doc test failure fixed. |
The current error is due to jaxlib 0.4.32 being yanked. If you rebase on the updated main branch, then tests should pass. Thanks! |
b2ab9d8
to
8db394e
Compare
rebased from the updated main branch. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, a couple more tweaks
description_changed_examp_added doc_byte_fixed discription_modified
8db394e
to
7dde9b2
Compare
@jakevdp |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks!
Part of #21461