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

Ring of springs integration test using modulo #1521

Merged
merged 4 commits into from
Nov 8, 2023
Merged

Conversation

martinjm97
Copy link
Collaborator

This PR is a slight change to the simplified springs example that changes the topology of the springs so that is a ring of 1D springs rather than a line of springs. Executing this code produced the following result on my machine:

Assertion failed: (abs(val) > 0.00001), function inner_store, file new_sparse.cpp, line 68.
[1]    24375 abort      ./new_sparse.out

@martinjm97
Copy link
Collaborator Author

martinjm97 commented Nov 8, 2023

The requested changes (most of which you made in our meeting :D) are in the updated branch. The dense code now works, but we run into issues for the auto-sparsified code.

Assertion failed: (i_nocapture < OperandTraits::operands(this) && "getOperand() out of range!"), function getOperand, file Instructions.h, line 2956.

@wsmoses wsmoses merged commit 82318b8 into EnzymeAD:main Nov 8, 2023
41 of 54 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants