DeepSpeed icon indicating copy to clipboard operation
DeepSpeed copied to clipboard

Published 20 hours ago • verified publisher icon microsoft

[XPU] support op builder from intel_extension_for_pytorch kernel path

Open YizhouZ opened this issue 1 year ago • 6 comments

#Motivation From our next release, xpu DeepSpeed related kernels would be put into intel_extension_for_pytorch. This PR is to add new op builders and use kernel path from intel_extension_for_pytorch. More ops like MOE and WOQ will be added.

YizhouZ avatar Apr 17 '24 03:04 YizhouZ

@loadams @mrwyattii Hi, could you help to trigger a CI for this PR? thanks!

YizhouZ avatar May 09 '24 05:05 YizhouZ

@loadams @mrwyattii Hi, could you help to trigger a CI for this PR? thanks!

Done @YizhouZ - could you run the pre-commit formatter to pass the formatting check? Thanks

loadams avatar May 09 '24 15:05 loadams

@loadams @mrwyattii Hi, could you help to trigger a CI for this PR? thanks!

Done @YizhouZ - could you run the pre-commit formatter to pass the formatting check? Thanks

Fixed! Could you run it again? Thanks.

YizhouZ avatar May 13 '24 06:05 YizhouZ

@tjruwase @loadams It seems nv-torch-latest-v100 / unit-tests tests failed somehow in today's merging commit. I did not see any error msg in log, is it something related to CI server?

YizhouZ avatar May 22 '24 03:05 YizhouZ

@tjruwase @loadams It seems nv-torch-latest-v100 / unit-tests tests failed somehow in today's merging commit. I did not see any error msg in log, is it something related to CI server?

@YizhouZ - this looks to be a transient error on the CI nodes, it should be fixed now.

loadams avatar May 22 '24 15:05 loadams

@tjruwase @loadams It seems nv-torch-latest-v100 / unit-tests tests failed somehow in today's merging commit. I did not see any error msg in log, is it something related to CI server?

@YizhouZ - this looks to be a transient error on the CI nodes, it should be fixed now.

thanks! could you help to trigger CI?

YizhouZ avatar May 23 '24 06:05 YizhouZ

Thanks for triggering CI. Do you have comments on this PR? @loadams @tjruwase

YizhouZ avatar May 26 '24 14:05 YizhouZ

Thanks for triggering CI. Do you have comments on this PR? @loadams @tjruwase

Hi @loadams @tjruwase, this PR seems not in the merge queue. Could you give us some suggestions for this PR? Thanks!

YizhouZ avatar Jun 05 '24 04:06 YizhouZ

Thanks for triggering CI. Do you have comments on this PR? @loadams @tjruwase

Hi @loadams @tjruwase, this PR seems not in the merge queue. Could you give us some suggestions for this PR? Thanks!

Hi @YizhouZ - we just haven't had time to review yet, we will prioritize this.

loadams avatar Jun 05 '24 15:06 loadams

Could you please add this PR into merge queue? Thanks! @tjruwase @loadams

YizhouZ avatar Jun 20 '24 08:06 YizhouZ