-
Notifications
You must be signed in to change notification settings - Fork 8
Add Support for GetAttribute and Indirect Access to Einsum Interpreter #230
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
TheRealMichaelWang
wants to merge
50
commits into
main
Choose a base branch
from
mw/add-insum-interpreter-support
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 37 commits
Commits
Show all changes
50 commits
Select commit
Hold shift + click to select a range
193dbb1
* Added basic sparse tensor type.
TheRealMichaelWang 101caf3
* Fixed ruff errors
TheRealMichaelWang ae38183
* Fixed mypy typing issues
TheRealMichaelWang bddbac9
* Fixed ruff whitespace errors
TheRealMichaelWang 7318931
* Added in dimension check for safety
TheRealMichaelWang 3d27adc
Merge branch 'main' of https://github.com/finch-tensor/finch-tensor-l…
TheRealMichaelWang 335c75c
* Added GetAttribute Einsum IR node
TheRealMichaelWang 6ab5e06
* Added support for printing GetAttribute Einsum IR node
TheRealMichaelWang e176ef4
* Added einsum interpreter support for evaluating GetAttribute IR node
TheRealMichaelWang 2936eac
* Added support for indirection in ein.Access
TheRealMichaelWang 2c061ab
* Added support for indirection with mutli-dimension indicies. I.e. A…
TheRealMichaelWang 46fda9c
* Added comments explaining indirect access implementation
TheRealMichaelWang d32ae5b
* Fixed bugs in einsum interpreter that stemmed from newly added supp…
TheRealMichaelWang 5f09be4
* Fixed some bugs
TheRealMichaelWang 860e016
* Revert changes to ein.Access
TheRealMichaelWang bf43dbb
* Added seperate match case handlers in einsum interpreter loop to ha…
TheRealMichaelWang 2f14a25
* Added support for one index indirect access
TheRealMichaelWang 9c57374
* Added support for multiple indirect indicies in access in einsum in…
TheRealMichaelWang 5ea1ddd
Enhanced einsum interpreter to evaluate tensor shapes and permute dim…
TheRealMichaelWang 71268a8
* Added match case in einsum interpreter loop to hand einsums with in…
TheRealMichaelWang 24ef52c
* Added support for getting shape attribute of a sparse tensor
TheRealMichaelWang 85d81f7
Implemented direct einsum handling for indirect assignments without r…
TheRealMichaelWang 710a2e3
Refactored indirect einsum handling in the interpreter to support red…
TheRealMichaelWang 1dc9667
Removed implementation of indirect einsum with reduction in the inter…
TheRealMichaelWang d953757
Enhanced EinsumInterpreter to handle cases with fewer indices than di…
TheRealMichaelWang 09c9879
Renamed test for indirect access to clarify focus on elementwise mult…
TheRealMichaelWang f65e4f9
Add tests for indirect access in einsum operations
TheRealMichaelWang 7549b62
Refactor EinsumInterpreter to handle flat indexing for 1D arrays and …
TheRealMichaelWang 29f5f03
Refactor EinsumInterpreter to improve handling of indirect indexing a…
TheRealMichaelWang 6fe0528
Remove redundant calculation of parent indices in EinsumInterpreter t…
TheRealMichaelWang 8d0a788
Refactor EinsumInterpreter to enhance indirect indexing logic by sepa…
TheRealMichaelWang cc09bf5
Refactor EinsumInterpreter to improve index handling by grouping ein.…
TheRealMichaelWang 169cebf
Refactor EinsumInterpreter to enhance index classification and groupi…
TheRealMichaelWang b2f81e6
* Fixed ruff errors
TheRealMichaelWang c31f7af
* Fixed mypy issues
TheRealMichaelWang d6f2d25
* Renamed GetAttribute to GetAttr to be consistent with FinchAssembl…
TheRealMichaelWang 9025903
Merge branch 'main' into mw/add-insum-interpreter-support
TheRealMichaelWang b59cc0a
Refactor EinsumInterpreter to streamline index evaluation and groupin…
TheRealMichaelWang 5d0b80b
* Fixed ruff errors
TheRealMichaelWang 3fe28f7
Merge branch 'main' into mw/add-insum-interpreter-support
willow-ahrens b387a0a
Merge branch 'main' of https://github.com/finch-tensor/finch-tensor-l…
TheRealMichaelWang 2a3adec
Merge branch 'mw/add-insum-interpreter-support' of https://github.com…
TheRealMichaelWang 64bfe6e
* Fixed minor issues in einsum node.py to adjust for new base einsum …
TheRealMichaelWang 8e69d16
* Changed indirect access implementation to a recursive approach
TheRealMichaelWang 0e527c7
Rewrote ein.Access implementation to access tensors with a mixture of…
TheRealMichaelWang f163f65
* Added two tests to einsum tests
TheRealMichaelWang 656aabe
* Added comments to enhance clarity in ein.Access for multiple indicies
TheRealMichaelWang 4bf1a78
* Simplified calculation of dest_axes in ein.Access
TheRealMichaelWang bd8e22c
* Fixed ruff errors
TheRealMichaelWang dd0167d
* Fixed end of file issues in .gitignore
TheRealMichaelWang 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
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.
Uh oh!
There was an error while loading. Please reload this page.