-
Notifications
You must be signed in to change notification settings - Fork 0
feat: flash attention support for hexagon-npu #16
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
Merged
Merged
Changes from all commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
ce39ca2
add params
chraac c33103e
add flash attention param
chraac bf16adb
wip
chraac 97d099c
fix bench param
chraac a2cc441
update how to build
chraac ecfe2da
add log parser
chraac ee6731e
update log parser regex and add tests for FLASH_ATTN_EXT properties
chraac ed61259
refactor log parser to improve readability and add list_from_iterable…
chraac a3a91c9
add support for parsing and saving log data to CSV; enhance OpItem wi…
chraac d095a1e
enhance log parser to support file encoding detection; refactor item …
chraac bbd4b4f
enhance log parser with detailed logging for parsing and saving opera…
chraac 06fef0f
wip
chraac 1439de8
bump llama.cpp version
chraac 3e54892
fix comments
chraac 53377ec
wip
chraac 07d855a
bump llama.cpp version
chraac 28fd500
wip
chraac 119bfd5
bump llama.cpp version
chraac 09a9f1e
add Python CI workflow and requirements for testing
chraac 63c8829
wip
chraac a850ed4
Revert "wip"
chraac 48832ed
update build script references in documentation
chraac File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,41 @@ | ||
# This workflow will install Python dependencies, run tests and lint with a single version of Python | ||
# For more information see: https://docs.github.com/en/actions/automating-builds-and-tests/building-and-testing-python | ||
|
||
name: Python unittests | ||
|
||
on: | ||
push: | ||
branches: | ||
- main | ||
pull_request: | ||
branches: | ||
- main | ||
|
||
permissions: | ||
contents: read | ||
|
||
jobs: | ||
python-unittest-scripts: | ||
runs-on: ubuntu-latest | ||
steps: | ||
- uses: actions/checkout@v3 | ||
- name: Set up Python 3.11 | ||
uses: actions/setup-python@v3 | ||
with: | ||
python-version: "3.11" | ||
- name: Install dependencies | ||
run: | | ||
python -m pip install --upgrade pip | ||
pip install flake8 pytest | ||
pip install -r ${GITHUB_WORKSPACE}/scripts/requirements.txt | ||
- name: Lint with flake8 | ||
run: | | ||
# stop the build if there are Python syntax errors or undefined names | ||
flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics | ||
# exit-zero treats all errors as warnings. The GitHub editor is 127 chars wide | ||
flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127 --statistics | ||
- name: Test with pytest | ||
run: | | ||
export PYTHONPATH="$PYTHONPATH:${GITHUB_WORKSPACE}:${GITHUB_WORKSPACE}/scripts:${GITHUB_WORKSPACE}/scripts/tests" | ||
cd scripts/tests | ||
python3 -m test_log_parser -v |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,3 +6,4 @@ build_qnn_* | |
temp/* | ||
*.txt | ||
run_logs | ||
*.pyc |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Empty file.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
The variable
$extraArgs
is only set inside theif ($Verbose)
block and may be undefined here. Initialize$extraArgs = ""
before any conditionals to prevent null or concatenation errors.Copilot uses AI. Check for mistakes.