-
Notifications
You must be signed in to change notification settings - Fork 126
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
[Enh]: Add support for Binary
& Time
datatype
#1989
Comments
@FBruzzesi another one that I noted in Feel free to move this to another issue, just wanted to collect some knowledge on missing types: General
Implementations
|
thanks both I think we should have both data types - even if we can't always support casting to it for all backends, if these datatypes are present, we should at least reported them as |
Thanks for pointing that out @dangotbanned , I will edit the issue in a bit to be more comprehensive for all the missing types |
Thanks @MarcoGorelli, @FBruzzesi Just to argue against myself a little bit on # nw.Time: "time" (Not Implemented, but we don't have any cases using it anyway) Per #1942:
I think it makes sense to support, but I'm not losing any sleep over the status quo 🙂 |
I think this fine, a couple new data types doesn't really add much bloat (as opposed to, say, implementing an entire new namespace) |
Binary
datatypeBinary
& Time
datatype
Description
We have support for the large majority of polars dtypes, yet missing a few:
Binary
Time
The text was updated successfully, but these errors were encountered: