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

Implements tree arithmetic for tensor add and multiplication #2024

Merged
merged 3 commits into from
Mar 6, 2021

Conversation

wjsi
Copy link
Member

@wjsi wjsi commented Mar 5, 2021

What do these changes do?

Implements tree arithmetic for tensor add and multiplication to make operand tree of consecutive arithmetic operations shallower. Most of the codes moved from mars.tensor.special.core.

Related issue number

Deal with #2023

@wjsi wjsi added mod: tensor type: feature New feature to be backported Indicate that the PR need to be backported to stable branch labels Mar 5, 2021
@wjsi wjsi added this to the v0.7.0a7 milestone Mar 5, 2021
Copy link
Collaborator

@qinxuye qinxuye left a comment

Choose a reason for hiding this comment

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

LGTM

@qinxuye qinxuye merged commit ab5283d into mars-project:master Mar 6, 2021
qinxuye pushed a commit to qinxuye/mars that referenced this pull request Mar 6, 2021
@wjsi wjsi deleted the enh/tree_arith_no_opt branch March 7, 2021 06:08
@qinxuye qinxuye added backported already PR has been backported and removed to be backported Indicate that the PR need to be backported to stable branch labels Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backported already PR has been backported mod: tensor type: feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants