-
Notifications
You must be signed in to change notification settings - Fork 63
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
Number proxies #250
Draft
jjsjann123
wants to merge
87
commits into
Lightning-AI:main
Choose a base branch
from
jjsjann123:number_proxies
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Number proxies #250
Changes from 7 commits
Commits
Show all changes
87 commits
Select commit
Hold shift + click to select a range
81abdf9
quick enabling proxy number
jjsjann123 4760d5a
disabling symbolic shape assert
jjsjann123 9a2ae70
hacky entry
jjsjann123 7dd515e
errr
jjsjann123 eb62a00
does it work for shapes?
jjsjann123 5b6ff2d
missing elif
jjsjann123 7ea52db
note
jjsjann123 9e4062c
Merge remote-tracking branch 'origin/main' into HEAD
jjsjann123 09f2405
trying out to remove Number type from NumberProxy
jjsjann123 99269b6
quick fix
jjsjann123 f1d4bd1
relaxing check_valid_length
jjsjann123 050e8bc
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 0a2bdd8
type not defined
jjsjann123 e81ebbf
patching
jjsjann123 1a2b6d3
fix WIP
jjsjann123 caa892b
comment out circular import
jjsjann123 6846ef4
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 428eebb
patch
jjsjann123 9205c60
Merge remote-tracking branch 'jiej/number_proxies_is_not_a_number' in…
jjsjann123 4fd4c2e
fixing more tests
jjsjann123 894ba4e
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] c930a11
import
jjsjann123 b373d8b
import
jjsjann123 89ba061
quick fixing tests
jjsjann123 77cb00f
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b142702
fixing grad maybe
jjsjann123 23ec660
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f414c21
fix
jjsjann123 a881996
fixing grad
jjsjann123 8cd2987
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b36bdf6
moving grad rule
jjsjann123 06cbd60
questionable patch
jjsjann123 5b5763a
Merge remote-tracking branch 'origin/main' into number_proxies_is_not…
jjsjann123 d154495
quick patching
jjsjann123 f3af55c
Merge branch 'main' into number_proxies_is_not_a_number
t-vi 435c086
cleaning
jjsjann123 0f19b66
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] a3413fd
more cleaning
jjsjann123 c375abf
quick hack to make CI green
jjsjann123 4620b39
Merge remote-tracking branch 'jiej/number_proxies_is_not_a_number' in…
jjsjann123 6b9ac97
I think I'm fixing it
jjsjann123 0a4cf9b
Merge remote-tracking branch 'jiej/number_proxies_is_not_a_number' in…
jjsjann123 75a0ba5
Merge branch 'main' into number_proxies
jjsjann123 9cf08aa
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] fb1b54d
Merge branch 'main' into number_proxies
jjsjann123 07dc56a
adding number like checks
jjsjann123 ecd7510
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] e37a311
quick fix on build
jjsjann123 0e607b2
Merge remote-tracking branch 'jiej/number_proxies' into number_proxies
jjsjann123 04cc9ce
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 688ce9f
disable implicit evaluation
jjsjann123 491ddb3
Merge remote-tracking branch 'jiej/disable_implicit_evaluate_of_numbe…
jjsjann123 c20e166
smoke test
jjsjann123 0f6cf9b
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f1cd374
fixing smoke test
jjsjann123 e549dbd
fixing reshape in nvfuserex
jjsjann123 4fdd912
Merge remote-tracking branch 'jiej/number_proxies' into number_proxies
jjsjann123 5c84029
quick fix
jjsjann123 e26417d
fixing nvfuser handling of sequence
jjsjann123 206f6a9
unwrap number proxy
jjsjann123 a0daeeb
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 0274f00
clean up
jjsjann123 b200382
error message for regex
jjsjann123 9bfac26
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] a5e4f41
fixing fill_value check
jjsjann123 ec369dd
Merge remote-tracking branch 'jiej/number_proxies' into number_proxies
jjsjann123 1c00ccd
import fix
jjsjann123 0aac6a2
import
jjsjann123 1c76716
fixing flip check
jjsjann123 b1ca5a1
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 4cb290b
quick patch
jjsjann123 b321d54
Merge remote-tracking branch 'jiej/number_proxies' into number_proxies
jjsjann123 3c87a64
quick patch
jjsjann123 6cdd428
fixing return
jjsjann123 760fc1f
fixing philox (mostly hacky patches)
jjsjann123 6cfcda3
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] fa50267
typo
jjsjann123 0abab1b
typo
jjsjann123 781329e
fixing tests
jjsjann123 a804552
Merge branch 'proxify_patch' into number_proxies
jjsjann123 901897f
Merge remote-tracking branch 'jiej/number_proxies' into number_proxies
jjsjann123 19a34ea
fixing numberproxy trace mode check
jjsjann123 775aa74
Merge remote-tracking branch 'origin/main' into number_proxies
jjsjann123 597e873
Merge remote-tracking branch 'origin/main' into HEAD
jjsjann123 79a9884
fixing typo
jjsjann123 8fe43a9
back out from dynamic shape
jjsjann123 fe6f311
Merge branch 'main' into number_proxies
jjsjann123 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.
What is our medium term plan w.r.t. defaul caching? If we need this for correctly handling #231 , it would seem that symbolic values should be the default but that in turn would mean that we want to have it work for our supported use cases.
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'm hoping that #231 wouldn't need to the whole
enable CACHE_OPTIONS.SYMBOLIC_VALUES
thing. Looks like #231 has passed all CI, which feels promising.The first step of this is to get number proxies to be plumbed through, I think that might be helpful for #231.
In terms of dynamic shape, we can slowly expanding its support.
Start off with allowing scalar input as number proxies, while still requiring tensor to be constant shape. I'm trying to figure out how/where to properly insert prologue_trace guard. Right now considering doing that from executor. i.e. nvfuser would require reduction dim(s) to be baked in as constant, while torchex doesn't care.
I'll try to come up with a design doc for review.