Skip to content

Conversation

shewu-quic
Copy link
Collaborator

@shewu-quic shewu-quic commented Aug 30, 2024

Summary:

QNN API version format is major.minor.patch.

  • If major given by the user does not match the built major, it will return and show error message.
  • If minor does not match, it will show warning message.

Scenarios

User QNN API Version Build QNN API Version Log Output
2.17.0 2.15.0 [WARNING] [Qnn ExecuTorch]: Qnn backend library version 2.17.0 is used. The version is tested against 2.15.0.
2.14.0 2.15.0 [WARNING] [Qnn ExecuTorch]: Qnn backend library version 2.14.0 is mismatched. The minimum supported version is 2.15.0. Please make sure you have the correct backend library version.
User QNN HTP Version Build QNN HTP Version Log Output
5.24.0 5.22.0 [WARNING] [Qnn ExecuTorch]: Backend kHtpBackend library version 5.24.0 is used. The version is tested against 5.22.0.
5.20.0 5.22.0 [WARNING] [Qnn ExecuTorch]: Backend kHtpBackend library version 5.20.0 is mismatched. The minimum supported version is 5.22.0. Please make sure you have the correct backend library version.

Author: yuyazhua

Summary:
QNN API version format is major.minor.patch.
If major given by the user does not match the built major, it will
return and show error message.
If minor does not match, it will show warning message.
Copy link

pytorch-bot bot commented Aug 30, 2024

🔗 Helpful Links

🧪 See artifacts and rendered test results at hud.pytorch.org/pr/pytorch/executorch/4998

Note: Links to docs will display an error until the docs builds have been completed.

✅ No Failures

As of commit b28db11 with merge base 369f804 (image):
💚 Looks good so far! There are no failures yet. 💚

This comment was automatically generated by Dr. CI and updates every 15 minutes.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 30, 2024
@shewu-quic
Copy link
Collaborator Author

Hi @cccclai,
The PR is to check the qnn sdk version given by the user and the built qnn sdk version.
Could you help to take a look?

Thanks!

Copy link
Contributor

@cccclai cccclai left a comment

Choose a reason for hiding this comment

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

This is super helpful. Thanks!

@cccclai cccclai merged commit b95e4b3 into pytorch:main Aug 30, 2024
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants