-
Notifications
You must be signed in to change notification settings - Fork 107
[Feature] Add more supports for NPU in addition to CUDA in previously supported use cases. #1471
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
base: main
Are you sure you want to change the base?
Conversation
vmoens
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR! Some comments but generally ok with it
vmoens
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this PR! Some comments but generally ok with it
|
@vmoens Could you please approve the workflow and this pull request? |
|
Do you have any problems with this PR? May it be merged? @vmoens |
|
Let's wait a bit for the tests to pass and I'll merge! Thanks for contributing |
|
Some tests failed but I think they are related to Win platform or some env problems? @vmoens |
|
sorry I was away last week, trying to fix a couple of things then we'll be good to go! |
Add more supports for NPU in addition to CUDA in previously supported use cases.
Description
Add more supports for NPU in addition to CUDA in previously supported use cases.
Types of changes
What types of changes does your code introduce? Remove all that do not apply:
Checklist
Go over all the following points, and put an
xin all the boxes that apply.If you are unsure about any of these, don't hesitate to ask. We are here to help!