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

add stack #3897

Merged
merged 8 commits into from
Dec 16, 2020
Merged

add stack #3897

merged 8 commits into from
Dec 16, 2020

Conversation

MARD1NO
Copy link
Contributor

@MARD1NO MARD1NO commented Nov 23, 2020

使用 expand_dims + concat 组合出stack算子

# Evaluate the gradient. Here we only test the reduction type == "mean"
assert np.allclose(blob, _np_grad)

# if the stack axis is 0, the batch axis will be change to 1
Copy link
Contributor Author

Choose a reason for hiding this comment

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

注:这里的处理逻辑是,如果stack的axis是0(也包括特殊负数情况,使得最终axis=0)那么stack后的tensor,它的batchaxis是会变化的,由0变为1,为了保证后续的multiply能够做正确的elementwise mul,要求两者的batch_axis一致,所以这里进行推断,用于 def oneflow_stack 中的 of_mul参数

@MARD1NO MARD1NO requested a review from Ldpe2G December 4, 2020 03:08
@Ldpe2G Ldpe2G marked this pull request as ready for review December 16, 2020 11:55
@oneflow-ci-bot oneflow-ci-bot removed their request for review December 16, 2020 13:19
@oneflow-ci-bot oneflow-ci-bot self-requested a review December 16, 2020 13:29
@oneflow-ci-bot oneflow-ci-bot merged commit 6941aa7 into master Dec 16, 2020
@oneflow-ci-bot oneflow-ci-bot deleted the dev_stack_python branch December 16, 2020 16:30
liujuncheng pushed a commit that referenced this pull request Jun 3, 2021
* add stack

* add grad check

* fix expression

* fix detail

Co-authored-by: Liang Depeng <liangdepeng@gmail.com>
Co-authored-by: oneflow-ci-bot <69100618+oneflow-ci-bot@users.noreply.github.com>
Former-commit-id: 6941aa7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants