Skip to content
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

[C++] Binary View Compute Kernels #44336

Open
WillAyd opened this issue Oct 8, 2024 · 1 comment
Open

[C++] Binary View Compute Kernels #44336

WillAyd opened this issue Oct 8, 2024 · 1 comment

Comments

@WillAyd
Copy link
Contributor

WillAyd commented Oct 8, 2024

Describe the enhancement requested

Is there a larger plan to start adding compute kernels for the binary view types? I see dedicated issues like #43010 but I'm not sure if there's a larger issue tracking these, to try and get parity with the (non-view) binary types

Component(s)

C++

@WillAyd
Copy link
Contributor Author

WillAyd commented Oct 10, 2024

I'd be happy to take a look at this in more detail. At a high level though, I wonder if StringView should be a part of the StringTypes defined in type.cc (ditto for Binary) or if we wanted to define that differently?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant