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

[build][cann] Fix CANN provider build failure #20698

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

shakhat
Copy link

@shakhat shakhat commented May 16, 2024

Description

This patch adds Python headers to CANN provider dependencies when building a library with both training and Python bindings.

Motivation and Context

Fixes #20697

This patch adds Python headers to CANN provider dependencies when
building a library with both training and Python bindings.
@jywu-msft
Copy link
Member

+@wangshuai09 , @zhangsibo1129
Is --enabling_training --use_cann a valid supported combination?

@wangshuai09
Copy link
Contributor

+@wangshuai09 , @zhangsibo1129 Is --enabling_training --use_cann a valid supported combination?
Sorry for the slow reply, the training ability is not adapted, so the --enabling_training is not supported.

@shakhat
Copy link
Author

shakhat commented May 26, 2024

+@wangshuai09 , @zhangsibo1129 Is --enabling_training --use_cann a valid supported combination?

Sorry for the slow reply, the training ability is not adapted, so the --enabling_training is not supported.

Ok, I see. My goal was to build one package that I can install in different environments (and CANN inference is one of them). Otherwise I have to build different artifacts for different use cases.

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.

[Build] [CANN] Failed to build CANN provider with training and Python bindings
3 participants