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 tensor grad assginment #5379

Merged
merged 67 commits into from Jul 7, 2021
Merged

Conversation

tea321000
Copy link
Contributor

@tea321000 tea321000 commented Jul 3, 2021

image

image

添加tensor.grad的setter,并补全zero_grad()的docstring和将slices添加到rst里。

BBuf and others added 30 commits June 29, 2021 21:41
Co-authored-by: oneflow-ci-bot <69100618+oneflow-ci-bot@users.noreply.github.com>
@oneflow-ci-bot oneflow-ci-bot self-requested a review July 6, 2021 05:05
@wyg1997
Copy link
Contributor

wyg1997 commented Jul 6, 2021

#5299 这个PR依赖Tensor.grad的setter方法

@oneflow-ci-bot oneflow-ci-bot removed their request for review July 6, 2021 06:41
@oneflow-ci-bot oneflow-ci-bot self-requested a review July 6, 2021 06:41
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 6, 2021 07:40
@oneflow-ci-bot oneflow-ci-bot removed their request for review July 6, 2021 09:24
@oneflow-ci-bot oneflow-ci-bot self-requested a review July 6, 2021 09:24
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 6, 2021 10:20
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 6, 2021 11:19
@wyg1997 wyg1997 removed the request for review from oneflow-ci-bot July 7, 2021 01:55
@oneflow-ci-bot oneflow-ci-bot requested review from oneflow-ci-bot and removed request for oneflow-ci-bot July 7, 2021 03:40
@oneflow-ci-bot oneflow-ci-bot merged commit bf1c827 into master Jul 7, 2021
@oneflow-ci-bot oneflow-ci-bot deleted the add_tensor_grad_assginment branch July 7, 2021 04:40
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.

None yet

4 participants