-
Couldn't load subscription status.
- Fork 701
Runner changes for TorchTune Llama3.2 vision text decoder #6610
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
46 commits
Select commit
Hold shift + click to select a range
7f81e00
Changes to native runner to run tt
jackzhxng 0b5a9a7
Add kwarg example inputs to eager model base
jackzhxng a9647d2
Create create new method for example kwarg inputs instead
jackzhxng fa3b1d2
Add kwarg example inputs to eager model base
jackzhxng e8715ba
Lint
jackzhxng a6f96a2
Accept model type parameter in export_llama
jackzhxng 328c72c
Remove future implementation
jackzhxng ec80bba
Lint
jackzhxng c9bbe12
Create create new method for example kwarg inputs instead
jackzhxng 99d5bfb
Accept model type parameter in export_llama
jackzhxng 1fb2236
Torchtune llama3_2_vision model in ET, no quantization
jackzhxng e0c4b8a
Fix vision model example input
jackzhxng e145bd1
Lint
jackzhxng ed906cb
Kv cache
jackzhxng 6dd47e7
Merge branch 'main' into jz/tt-llama
jackzhxng 1825972
Update READMEs
jackzhxng 196499a
Change model default arg
jackzhxng 96ba40b
Update eager runner and eval llama
jackzhxng 18a82e1
Merge branch 'jz/tt-llama-rebased' into jz/tt-llama-2
jackzhxng 0f3035d
Fix tests
jackzhxng e677e14
Merge branch 'jz/tt-llama-rebased' into jz/tt-llama-2
jackzhxng b1f6678
Fix tests again
jackzhxng 13d004b
Merge branch 'jz/tt-llama-rebased' into jz/tt-llama-2
jackzhxng c79b773
Strict = True
jackzhxng b8ff8e2
Things work
jackzhxng 25ec7ce
Merge branch 'jz/tt-llama-rebased' into jz/native-runner-tt
jackzhxng 6e38763
Clip logits if torchtune
jackzhxng 7a7041d
Merge branch 'jz/tt-llama-2' into jz/native-runner-tt
jackzhxng 96d5798
Fix
jackzhxng f275e2e
Kv cache by default is false
jackzhxng 37011d3
Clean up
jackzhxng de45c48
Strict = True
jackzhxng 2fe7bd8
Merge branch 'main' into jz/tt-llama-2
jackzhxng 64dcbda
Lint
jackzhxng a89d6b2
Fix merge
jackzhxng e1ec74c
Merge branch 'jz/tt-llama-2' into jz/native-runner-tt
jackzhxng 84422d9
Fixes
jackzhxng 1163769
Remove token count printing
jackzhxng e5428de
Move to subdir
jackzhxng eefadaa
Merge branch 'jz/tt-llama-2' into jz/native-runner-tt
jackzhxng bf33485
Merge remote-tracking branch 'origin/main' into jz/tt-llama-2
jackzhxng 9c5647c
Merge branch 'jz/tt-llama-2' into jz/native-runner-tt
jackzhxng f61a347
Tarun rev
jackzhxng 7a0101f
Add automatically generated export tests
jackzhxng 9777e23
Fix internal pyre warning
jackzhxng 2b9f281
Merge branch 'jz/tt-llama-2' into jz/native-runner-tt
jackzhxng 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
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
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
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.
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 we'd want the default to still be True?
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.
Oh yeah this was weird since "store_true" works by having a default of False, but the default is here set to True so it's just always True regardless of what you put