-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
Add tensor parallel inference unit tests #2232
Merged
Merged
Changes from all commits
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
770c59a
added mp test
mrwyattii 706617f
Merge branch 'master' into add-mp-inf-tests
mrwyattii c0925a1
make text-gen compare DS output to baseline
mrwyattii 36bb2c5
Merge branch 'master' into add-mp-inf-tests
mrwyattii f942b5a
Merge branch 'master' into add-mp-inf-tests
RezaYazdaniAminabadi adaabe0
remove non-gpt models from multigpu test
mrwyattii f9a7eae
add more models and fixes for OOM
mrwyattii 606a669
added job ID to environment printout
mrwyattii aef7603
made multi-GPU tests sequential
mrwyattii 27a9825
Merge branch 'master' into add-mp-inf-tests
mrwyattii 7cb4136
update inference workflow with sequential tests
mrwyattii 409af10
fix for multi-gpu
mrwyattii 28c8102
fix for selecting inference sequential tests
mrwyattii 185d4b9
change default to not include inference tests
mrwyattii 5c84dca
restrict gptneox to half precision
mrwyattii 6edb13b
increas world_size to 4 for multigpu
mrwyattii a75d5fe
Merge branch 'master' into add-mp-inf-tests
jeffra a79bb15
Merge branch 'master' into add-mp-inf-tests
jeffra 2ab79f0
Merge branch 'master' into add-mp-inf-tests
samadejacobs 19839cb
use fp16 revision of gpt-j
jeffra 776be03
revert non-gptj
jeffra 429c88d
use gpt-j fp16 revision
jeffra b6d2b3b
Merge branch 'master' into add-mp-inf-tests
jeffra af8d94d
Merge branch 'master' into add-mp-inf-tests
jeffra 335406a
fix for broken tests
mrwyattii d0bb608
fix invalid model/task check
mrwyattii 4678f05
correct gpt-neo test that should have been skipped
mrwyattii 5b5e65a
add exception for bloom models failing to match text-gen output
mrwyattii 19f2bf0
restrict number of tokens generated
mrwyattii 7feac82
fix for opt models
mrwyattii cfdf629
Update test_inference.py
mrwyattii 3e87d74
Merge branch 'master' into add-mp-inf-tests
mrwyattii 42ccea5
formatting
mrwyattii db48d24
Merge branch 'master' into add-mp-inf-tests
mrwyattii 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 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 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 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 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 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 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 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 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 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 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 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 |
---|---|---|
@@ -1,6 +1,7 @@ | ||
[pytest] | ||
addopts = -m "not sequential and not nightly and not inference" | ||
addopts = -m "not sequential and not nightly and not inference and not seq_inference" | ||
markers = | ||
sequential:Tests that need to be run sequentially | ||
inference:Inference model tests | ||
seq_inference:Inference model tests to run sequentially | ||
nightly:Tests that should be run nightly |
This file contains 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
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.
I think we can change to this be latest master from HF, after we merge: #2291. But I want to re-test #2291 after this PR is merged.