-
Notifications
You must be signed in to change notification settings - Fork 694
Qualcomm AI Engine Direct - docs fix #14881
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -111,12 +111,13 @@ This section outlines the essential APIs and utilities provided to streamline th | |
Creates a clean directory for storing model outputs or intermediate results. If the directory already exists, it will be deleted and recreated to ensure a consistent environment for each run. | ||
|
||
## Additional Dependency | ||
This example requires the following Python packages: | ||
- pandas and scikit-learn: used in the mobilebert multi-class text classification example. | ||
- graphviz (optional): used for visualizing QNN graphs during debugging. | ||
|
||
The mobilebert multi-class text classification example requires `pandas` and `sklearn`. | ||
Please install them by something like | ||
|
||
```bash | ||
pip install scikit-learn pandas | ||
pip install scikit-learn pandas graphviz | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I had to install There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Thanks, I see it is part of There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yeah I think it's reasonable to have it as part of the instructions to run llama models There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I’ve just added the instruction at this line |
||
``` | ||
|
||
## Limitation | ||
|
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.
@cccclai we need to update these instructions. We shouldnt need to download sdk with qnn pip wheel. Make this either optional or remove it entirely. I would prefer latter because it avoid confusion
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.
pip wheel release hasn't finished yet. The nightly build is based on
viable/strict
and it has been stuck for a week, which doesn't include our latest commit for the fix. After it's verified, I can add a session and move this part to build from source session