-
Notifications
You must be signed in to change notification settings - Fork 44
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
Bring back Python backend based PyTorch backend #117
Merged
Merged
Changes from 11 commits
Commits
Show all changes
13 commits
Select commit
Hold shift + click to select a range
d126b08
Add Python backend based PyTorch runtime
kthui 8d50071
Add exec env build
kthui 4381340
Add note for adding .pt2 model support
kthui f71dd17
Do not specify pytorch cuda version
kthui b459f73
Do not install Python runtime on non x86
kthui 8ff6a3a
Merge branch 'main' of github.com:triton-inference-server/pytorch_bac…
kthui 78c47fe
Remove legacy comment
kthui e7cf4d2
Merge branch 'main' of github.com:triton-inference-server/pytorch_bac…
kthui 8b856f6
User to build PyTorch env
kthui 7d8a3a7
Add docs
kthui 9aa6b41
Update copyright
kthui b8abcaa
Clarify model layout between PyTorch and TorchScript
kthui 63251ba
Fix header size
kthui 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
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.
I think this is probably a little bit misleading since we don't always require both
model.py
andmodel.pt
(e.g., if torchscript is provided onlymodel.pt
is required as you mentioned).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.
Good spotting! I created different sections for PyTorch 2.0 and TorchScript model layout. Clarify model layout between PyTorch and TorchScript