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

[HuaweiAscendNPU] Adjust the code to obtain Ascend CANN version #9885

Merged

Conversation

shentanyue
Copy link
Collaborator

No description provided.

@paddle-bot
Copy link

paddle-bot bot commented Dec 29, 2022

Thanks for your contribution!

}
std::regex express(
"version=([0-9]+\\.[0-9]+\\.(RC)?[0-9]+\\.(alpha)?[0-9]*)");
#if defined(LITE_WITH_ARM)
Copy link
Collaborator

Choose a reason for hiding this comment

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

为了保证 NNAdapter的独立性,LITE_WITH_ARM 类似任何lite的宏都不能用在 NNAdapter 内部,可以参考 https://github.com/hong19860320/PaddleLite-generic-demo/blob/2f4ea88e1408953b45469589fe5b678c5f4843db/image_classification_demo/shell/demo.cc#L585

Copy link
Collaborator

Choose a reason for hiding this comment

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

关于 LITE_WITH_PYTHON 的宏定义也建议用 NNAdapter 本身的宏,例如 NNADAPTER_WITH_PYTHON

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done

Copy link
Collaborator

@hong19860320 hong19860320 left a comment

Choose a reason for hiding this comment

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

LGTM

@shentanyue shentanyue merged commit 25fcf2c into PaddlePaddle:develop Jan 4, 2023
@shentanyue shentanyue deleted the upgrade_ascend_cann_version branch January 4, 2023 03:45
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