Skip to content

Register jagged_to_padded_dense bool dispatch#3111

Open
Silv3S wants to merge 2 commits intointel:mainfrom
Silv3S:jagged_to_padded_bool
Open

Register jagged_to_padded_dense bool dispatch#3111
Silv3S wants to merge 2 commits intointel:mainfrom
Silv3S:jagged_to_padded_bool

Conversation

@Silv3S
Copy link
Contributor

@Silv3S Silv3S commented Mar 20, 2026

Fixes #2422
This PR adds support for bool inputs in jagged_to_padded_dense op. dense_to_jagged already supports it.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds XPU support for bool inputs in aten::_jagged_to_padded_dense_forward by extending the type dispatch in the SYCL kernel implementation, aligning it with existing dense_to_jagged behavior and addressing issue #2422.

Changes:

  • Extend jagged_to_padded_dense_forward_xpu_kernel dispatch to include at::ScalarType::Bool.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[upstream_ut] NotImplementedError: "jagged_to_padded_dense_xpu" not implemented for 'Bool'

2 participants