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

[Fix] delete the error comment #514

Merged
merged 1 commit into from
Oct 8, 2022

Conversation

vansin
Copy link
Collaborator

@vansin vansin commented Sep 5, 2022

Motivation

the input is randn, so we should delete the comment # tensor([-1.5774, -0.5850])

import torch
input = torch.randn(2)
output = model(input)
# call RReLU.forward
print(output)
# tensor([-1.5774, -0.5850])

Modification

delete the error comment

@vansin vansin mentioned this pull request Sep 5, 2022
@ZwwWayne ZwwWayne added this to the 0.2.0 milestone Sep 6, 2022
@ZwwWayne ZwwWayne merged commit 1f63d24 into main Oct 8, 2022
austinmw pushed a commit to austinmw/mmengine that referenced this pull request Oct 8, 2022
ZwwWayne pushed a commit that referenced this pull request Oct 8, 2022
* Add smddp dist backend option

* [Dev]: Upgrade pre commit hooks (#576)

* Upgrade the versions of pre-commit-hooks

* update zh-cn.yaml

* [Docs] Fix the docstring of model sub-package (#573)

* [Doc]: Update config.md (#562)

* Update config.md

* Update config.md

* [Doc] delete the error comment  in docs (#514)

Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zhengfei-0311 <78833899+Zhengfei-0311@users.noreply.github.com>
Co-authored-by: vansin <msnode@163.com>
C1rN09 pushed a commit to C1rN09/mmengine that referenced this pull request Nov 1, 2022
* Add smddp dist backend option

* [Dev]: Upgrade pre commit hooks (open-mmlab#576)

* Upgrade the versions of pre-commit-hooks

* update zh-cn.yaml

* [Docs] Fix the docstring of model sub-package (open-mmlab#573)

* [Doc]: Update config.md (open-mmlab#562)

* Update config.md

* Update config.md

* [Doc] delete the error comment  in docs (open-mmlab#514)

Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zhengfei-0311 <78833899+Zhengfei-0311@users.noreply.github.com>
Co-authored-by: vansin <msnode@163.com>
@zhouzaida zhouzaida deleted the vansin/fix-docs-registry-error-comment branch November 9, 2022 06:43
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.

3 participants