-
Notifications
You must be signed in to change notification settings - Fork 801
[Pytorch] Enable TE Op to consume extra_outputs from a previously run Op in TE Sequential #3320
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
Open
vthumbe1503
wants to merge
39
commits into
NVIDIA:main
Choose a base branch
from
vthumbe1503:enable_extra_out_consumption
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.
Open
Changes from all commits
Commits
Show all changes
39 commits
Select commit
Hold shift + click to select a range
b1574f0
produce/consume extra output
vthumbe1503 63192ab
allow for fusions with producer/consumer being part of same fuser wit…
vthumbe1503 3b4b523
cleanup
vthumbe1503 de38ed8
minor cleanup
vthumbe1503 385b0d5
dispatch combine impl
vthumbe1503 ad3b044
fusible ops test
vthumbe1503 5fb0d3a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 2ba4f6a
Merge remote-tracking branch 'nvidia_origin/main' into enable_extra_o…
vthumbe1503 3af2ecc
keep just ops infra changes
vthumbe1503 d7d6380
cleanup with residual tests
vthumbe1503 74f563a
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 29d23f2
Merge branch 'main' into enable_extra_out_consumption
vthumbe1503 87e2b36
address review comment
vthumbe1503 80601dc
update to cleaner documentation
vthumbe1503 5070e34
address review comments
vthumbe1503 ae41ad3
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] f82cbed
some cleanup
vthumbe1503 5a4e1ec
update docs
vthumbe1503 0a479c7
pin channels through channel version
vthumbe1503 d679998
unecessary handling removal
vthumbe1503 8f7ba95
simplify
vthumbe1503 c62bb15
doc update + extra_grad = None case
vthumbe1503 a93b820
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] 35b73b1
test cleanup
vthumbe1503 6801a6d
no need to check staleness in every forward call
vthumbe1503 6688e8a
remove redundant tests
vthumbe1503 12430c2
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] b189550
revert from bad names
vthumbe1503 a4cc112
keep simple
vthumbe1503 7edaf89
Merge branch 'enable_extra_out_consumption' of github.com:vthumbe1503…
vthumbe1503 5ba6055
unecessary checks
vthumbe1503 76826dc
minor doc
vthumbe1503 827f8e9
Merge branch 'main' into enable_extra_out_consumption
vthumbe1503 63a4ea3
fix lint
vthumbe1503 87c1cf6
Update transformer_engine/pytorch/ops/fuser.py
vthumbe1503 307ab15
Update docs/examples/op_fuser/op_fuser.rst
vthumbe1503 97a91cf
Update transformer_engine/pytorch/ops/fuser.py
vthumbe1503 6468a14
address review comments + extra output being configurable to be outpu…
vthumbe1503 2eb21ed
cleanup
vthumbe1503 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
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.
It would be nice if
Sequentialcould handle channels acrossOperationFusers, but the implementation would be quite hairy and not worth it for the current effort.