-
Notifications
You must be signed in to change notification settings - Fork 531
Add bfloat16 extension array #1002
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
Conversation
4b74569 to
5e8e8f1
Compare
14cc4ea to
7e5966d
Compare
| from ml_dtypes import bfloat16 | ||
| from lance.arrow import PandasBFloat16Array, BFloat16Array | ||
|
|
||
| np_array = np.array([1.1, 2.1, 3.4], dtype=bfloat16) |
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.
Could you make sure that it works with tensorflow bf16?
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.
tensorflow converts to numpy with the ml_dtypes.bfloat16, so I think we are good here 👍
ff466a2 to
ba01bb7
Compare
ba01bb7 to
521fc09
Compare
eddyxu
left a comment
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.
LGTM.
please make sure every file has a license header.
Closes #994