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.

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

[NPU] Add IPEX_LLM_NPU_MTL to enable support on mtl #12543

Merged
merged 2 commits into from
Dec 13, 2024

Conversation

plusbang
Copy link
Contributor

@plusbang plusbang commented Dec 13, 2024

Description

Background: https://github.com/analytics-zoo/nano/issues/1785

2. User API changes

To run optimized models on MTL NPU, need to set following variable:

set IPEX_LLM_NPU_MTL=1

No change for LNL.

4. How to test?

  • Application test

@@ -90,6 +90,9 @@ For `ipex-llm` NPU support, set the following environment variable with active `

```cmd
set BIGDL_USE_NPU=1
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe we can rename to IPEX_LLM_USE_NPU later ? Now it's a little strange with other env vars.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Maybe we can rename to IPEX_LLM_USE_NPU later ? Now it's a little strange with other env vars.

Good catch! I think we could also verify whether this is still necessary later : )

Copy link
Contributor

@rnwang04 rnwang04 left a comment

Choose a reason for hiding this comment

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

others LGTM.

@plusbang plusbang merged commit caf15cc into intel:main Dec 13, 2024
1 check passed
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.

2 participants