Skip to content
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

Tensor broadcast #6

Open
fredrikbk opened this issue Feb 10, 2017 · 1 comment
Open

Tensor broadcast #6

fredrikbk opened this issue Feb 10, 2017 · 1 comment
Assignees
Labels
feature Indicates new feature requests

Comments

@fredrikbk
Copy link
Contributor

fredrikbk commented Feb 10, 2017

taco "A(i,j) = b(i)"
Error at /Users/fred/Dropbox/projects/tensor-compiler/taco/src/tensor.cpp:524 in setExpr:
 All variables must appear on the right-hand-side of an assignment. This restriction will be removed in the future.
Expression: A(i,j) = b(i)
Abort trap: 6
@fredrikbk fredrikbk added the bug Indicates an unexpected problem or unintended behavior label Feb 10, 2017
@fredrikbk fredrikbk changed the title a(i,j) = b(i) taco "a(i,j) = b(i)" Feb 10, 2017
@fredrikbk
Copy link
Contributor Author

fredrikbk commented Feb 21, 2017

The second loop in this expression (j) has an empty merge rule, because no inputs index into j. However, we must still iterate over j to copy b(i) into each column A(i,*). This loop must be dense.

Thus the rule: Empty merge rules mean dense iteration. But we need some way to say what dimension the dense iteration is over.

@fredrikbk fredrikbk added this to the Beta release milestone Mar 2, 2017
@fredrikbk fredrikbk self-assigned this Mar 2, 2017
@fredrikbk fredrikbk changed the title taco "a(i,j) = b(i)" taco "A(i,j) = b(i)" Mar 6, 2017
@fredrikbk fredrikbk changed the title taco "A(i,j) = b(i)" A(i,j) = b(i) Mar 11, 2017
@fredrikbk fredrikbk removed this from the Beta release milestone May 12, 2017
@fredrikbk fredrikbk changed the title A(i,j) = b(i) Tensor distribution May 14, 2017
@fredrikbk fredrikbk added feature Indicates new feature requests and removed bug Indicates an unexpected problem or unintended behavior labels May 14, 2017
@fredrikbk fredrikbk changed the title Tensor distribution Tensor broadcast Aug 3, 2017
rohany added a commit that referenced this issue Oct 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature Indicates new feature requests
Projects
None yet
Development

No branches or pull requests

1 participant