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.

Sign up for GitHub

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

Jump to bottom

"index_copy_xpu" not implemented for 'Float8_e4m3fn' #1336

Open
huaiyuzh opened this issue Feb 6, 2025 · 2 comments
Open

"index_copy_xpu" not implemented for 'Float8_e4m3fn' #1336

huaiyuzh opened this issue Feb 6, 2025 · 2 comments
Assignees
@xytintel @yucai-intel
Milestone
PT2.8

Comments

@huaiyuzh
Copy link
Contributor

huaiyuzh commented Feb 6, 2025

index_copy_xpu cause an IPEX UT fail. In IPEX2.6, we override this Ops with IPEX implementation to make this UT pass.

ipex/tests/gpu/example/test_fp8_index_copy.py::TestTorchMethod::test_index_copy - RuntimeError: "index_copy_xpu" not implemented for 'Float8_e4m3fn'
@xytintel xytintel added this to the PT2.8 milestone Feb 21, 2025
@xytintel
Copy link
Contributor

#1393

@yucai-intel
Copy link
Contributor

We are consistent with the datatype supported by cuda, which does not support Float8_e4m3fn in index_copy_cuda.

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

@xytintel xytintel

@yucai-intel yucai-intel

Labels
None yet
Projects
None yet
Milestone
PT2.8
Development

No branches or pull requests
3 participants
@xytintel @huaiyuzh @yucai-intel