-
Notifications
You must be signed in to change notification settings - Fork 24.8k
[associative_scan] Autograd separated #139939
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
bohnstingl
wants to merge
34
commits into
pytorch:main
Choose a base branch
from
bohnstingl:associative_scan_74
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.
+616
−110
Open
Changes from 1 commit
Commits
Show all changes
34 commits
Select commit
Hold shift + click to select a range
9c49a36
WIP: Associative_scan Autograd
bohnstingl 100c598
Working implementation of Autograd
bohnstingl 0e7c8d5
Added partial gradient tests
bohnstingl 67d62fb
Separated out the partial gradient functionality to a separate PR
bohnstingl 9e5e1f9
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl 6b41565
Working version uncleaned
bohnstingl d68b31b
Almost all tests pass
bohnstingl 9de0caf
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl 653eab0
First working implementation of simplified autograd for combine_mode=…
bohnstingl ac1a12b
Merge branch 'associative_scan_74' of github.com:bohnstingl/pytorch i…
bohnstingl d51b16c
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl dbfd544
WIP: No additional_input support yet
bohnstingl 64a8b29
Updates and cosmetic fixes
bohnstingl 08b7251
iFixed problem with python<3.11
bohnstingl f378827
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl b12777e
Fixed issue with adding tuple and list
bohnstingl 6335022
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl 1c9fa70
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl 6d8353b
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl 3f81639
Rework to improve readability and unify shared function with scan
bohnstingl 62770d5
Removed irrelevant testcases, improved readability, extended document…
bohnstingl 6761714
Restructured documentation and synced with associative_scan
bohnstingl b73c553
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl ea9568d
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl 0786e6f
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl c628dd8
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl e7d63b6
Skipping autograd test for associative_scan with lifted arguments
bohnstingl 95d0e42
Reworked documentation
bohnstingl 3707c0d
Factored failing test out into separate test
bohnstingl a565834
Rework of documentation
bohnstingl 122d70b
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl d89bca6
Merge branch 'main' of github.com:pytorch/pytorch into associative_sc…
bohnstingl c1f1267
Removed former failing testcase for associative_scan
bohnstingl a59ad4c
Consolidated utility functions between scan.py and associative_scan.p…
bohnstingl 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
Factored failing test out into separate test
- Loading branch information
commit 3707c0df1bdc33a0f4d9fea2d2d74f11bcdec9a0
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.
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 separated off these two specific cases which lead to CPP compilation failures, which I don't think are necessarily related to the
associative_scan
, as they appear only here. I marked those tests as expected fail and we should do it in a follow-up PR. For reference, the issue we observe with those tests is: