Skip to content

Conversation

gausshj
Copy link
Collaborator

@gausshj gausshj commented Sep 12, 2025

  • Add assertion tests for matmul of impure tensors

Copy link
Member

@hzhangxyz hzhangxyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个pr整体没啥毛病,impure even的参数少点吧,broadcast, x, impure even是乘起来的数目,很大了。

torch.randn([*[x + impure_even for x in broadcast_b], dim_common, dim_b]),
).update_mask()
if a.tensor.dim() <= 2 and b.tensor.dim() <= 2:
pytest.skip("One of the two tensors needs to have a dimension greater than 2")
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里真的skip了什么东西么?应该没有吧?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

因为后面判断了AssertionError:“All edges except the last two must be pure even”,所以张量围度必须大于2。

Copy link

codecov bot commented Oct 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

- Add assertion tests for matmul of impure tensors

Signed-off-by: Gausshj <kylin_0@qq.com>
Signed-off-by: Hao Zhang <hzhangxyz@outlook.com>
hzhangxyz
hzhangxyz previously approved these changes Oct 9, 2025
@hzhangxyz hzhangxyz force-pushed the test/add-more-test-for-matmul branch from 05e3a47 to 7ded7f1 Compare October 9, 2025 11:14
@hzhangxyz hzhangxyz merged commit eed595c into main Oct 9, 2025
33 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