-
Notifications
You must be signed in to change notification settings - Fork 117
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
refine Dockerfile to support both cpu and xpu platform #981
Conversation
Signed-off-by: Liu, Kaixuan <[email protected]>
@echarlaix , Can you help review? Thx. |
Signed-off-by: Liu, Kaixuan <[email protected]>
The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update. |
There's a CI workflow that tests building the image, can you update it as well ? |
Oh, yes. have fixed it. |
If this Dockerfile in the is specificially for IPEX, should it be renamed to Dockerfile.ipex, to not confuse users? |
Signed-off-by: Liu, Kaixuan <[email protected]>
@helena-intel ,Hi, this Dockerfile is just used for CI test, but I think your advice is OK. Have renamed it to |
Hi ! I aslo think having separate Docker files is better. The build test is failing tho |
Signed-off-by: Liu, Kaixuan <[email protected]>
Seems there is sth wrong with access to machine putting release wheel files. Have replaced --extra-index-url from |
Signed-off-by: Liu, Kaixuan <[email protected]>
No description provided.