forked from pytorch/pytorch
-
Notifications
You must be signed in to change notification settings - Fork 7
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
IterDomain resize for pad, cat, slice #2480
Merged
Merged
Changes from 81 commits
Commits
Show all changes
93 commits
Select commit
Hold shift + click to select a range
a212e8d
WIP
naoyam 8d66afa
WIP: expand
naoyam e5ee102
WIP
naoyam 4b707e8
WIP
naoyam 66e5e38
v1
naoyam b458177
wip
naoyam 179679c
tests
naoyam c27900d
WIP: PadOp
naoyam ba6d5ef
checkpoint: decided to try rfactor
naoyam b826fbc
pad test
naoyam 8267442
computeAt
naoyam 395939d
parallelize
naoyam a7f42d2
concat
naoyam c748580
cat tests
naoyam f823b26
WIP: slice
naoyam 0251bad
bug fix
naoyam 5aed8ec
cleanup
naoyam c67927e
Merge branch 'devel' into catop
naoyam 49195d4
Merge branch 'devel' into catop
naoyam c7f2118
Renamed expand to resize
naoyam ecb72a4
Merge branch 'devel' into catop
naoyam 0f74cc8
cleanup
naoyam 71a99d3
Merge branch 'devel' into catop
naoyam 1864ec9
cleanup
naoyam 32e7d39
cleanup
naoyam 36f31e0
remove TensorView::expand
naoyam 4b5e3da
cleanup
naoyam e23b0c9
fix
naoyam 071f7d2
cleanup
naoyam 03f3194
cleanup
naoyam 13bb33a
cleanup
naoyam 4374489
cleanup
naoyam 5164c41
update
naoyam b056a86
cleanup
naoyam 1871d7f
fix
naoyam 27204dd
Merge branch 'devel' into iter_domain_resize
naoyam b4f24a4
test cleanup
naoyam bf21fb2
cleanup
naoyam bdc01ab
unswitch fix
naoyam d32a05f
updates for resize
naoyam dc3a4ee
more tests
naoyam 497f330
add scheduler tests for cat
naoyam 1faae1e
Don't propagate resize if not safe
naoyam 0238749
update
naoyam c792e41
WIP: scheduler support
naoyam 00f4a88
cleanup
naoyam 66d75b3
fix
naoyam bf45b91
fix
naoyam 594f50d
Softmax + slice working
naoyam aa4c356
cleanup, should be more robust
naoyam b036389
cleanup
naoyam 6701006
All tests pass
naoyam bcb3c8f
fix
naoyam 79e035c
cleanup
naoyam c492d57
cleanup
naoyam 9c2f350
fix
naoyam 2b8bb4d
cleanup
naoyam 01df3c8
Merge branch 'devel' into iter_domain_resize
naoyam d8534ca
cleanup
naoyam 460ad43
cleanup
naoyam 4a31361
cleanup
naoyam c79a4cd
Merge branch 'devel' into iter_domain_resize
naoyam 1665acd
cleanup
naoyam 152c01a
cleanup
naoyam 5e558e9
Merge branch 'devel' into iter_domain_resize
naoyam b476743
cleanup
naoyam 64c5bd1
PR feedback
naoyam 2a2e189
Mark pad widths of PadOp as inputs
naoyam 30712e7
cleanup
naoyam 4ad0b9d
comments
naoyam 01722c7
cleanpu
naoyam c4d6b39
Rename "disjoint view" to "disjoint rfactor"
naoyam 27c867f
Set the producer and consumer maps of the ID graph using the
naoyam 622ee93
Do not create index maps for non-indexable domains
naoyam c4aab87
cleanup
naoyam b0ffb91
fix dependency analysis
naoyam da7feb1
Naming fix
naoyam 8ad222c
Use pad instead of manually creating PadOp
naoyam 1b5c23c
Remove unused func
naoyam 6901da3
remove unnecessary func
naoyam 2264406
PR feedback
naoyam a73ff45
Clean up pad_width vector
naoyam b3cffb2
Convert pad to set when valid
naoyam 26061eb
More descriptive error messages
naoyam 232e701
Merge branch 'devel' into iter_domain_resize
naoyam f187d6d
Missing override implementation
naoyam c1d2be5
test fix
naoyam 3ed4797
Add validation of resize usage
naoyam 82845fa
rename test file
naoyam 1e2c711
fix
naoyam e366272
Merge branch 'devel' into iter_domain_resize
naoyam 8c1e228
merge fix
naoyam 39e603e
test fix
naoyam 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
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.
Am I understanding correct that, if I have
I1
andI3
exact mapped, andIf
l1->sameAs(l2) && r1->sameAs(r2)
, thenexact map, permissive map:
permissive resize map:
And if
!(l1->sameAs(l2) && r1->sameAs(r2))
, thenexact map, permissive map:
permissive resize map:
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 not sure what you mean by
producer: I2
andconsumer: I4
, but assuming they are just produced fromI1
andI3
, respectively, yes, that is correct.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 meant I have
where
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.
Assuming
transformations: I2 = resize(I1, l1, r1)
is not done as part of the rfactor transformation ofTproducer
, it only happens whenTproducer
is transformed asTconsumer
for producer indexing. In that case,whileI2
andI4
should map, we don't update the ComputeAt map, so it won't affect how any of the exact/permissive/etc maps.