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

[DOCS] Fix doc typos in MMDetection 3.x(useful tools) #9177

Merged
merged 19 commits into from
Nov 23, 2022

Conversation

sanbuphy
Copy link
Contributor

@sanbuphy sanbuphy commented Oct 29, 2022

I will fix typos in

doc name en version zh_cn version
useful_tools.md (all)
useful_hooks.md (all)

@sanbuphy sanbuphy marked this pull request as draft October 29, 2022 01:37
@ZwwWayne ZwwWayne added the v-3.x label Nov 2, 2022
@sanbuphy sanbuphy marked this pull request as ready for review November 3, 2022 09:56
@sanbuphy
Copy link
Contributor Author

sanbuphy commented Nov 7, 2022

@ZwwWayne I have finished the all jobs about useful_tools.md

docs/zh_cn/user_guides/useful_hooks.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_hooks.md Show resolved Hide resolved
docs/zh_cn/user_guides/useful_hooks.md Outdated Show resolved Hide resolved
@ZwwWayne ZwwWayne added this to the 3.0.0rc4 milestone Nov 7, 2022
@BIGWangYuDong
Copy link
Collaborator

Hi, thanks for your kind PR, please fix your lint. You can refer to https://github.com/open-mmlab/mmcv/blob/2.x/CONTRIBUTING.md to install pre-commit

@sanbuphy
Copy link
Contributor Author

Hi, thanks for your kind PR, please fix your lint. You can refer to https://github.com/open-mmlab/mmcv/blob/2.x/CONTRIBUTING.md to install pre-commit

OK, get it .

docs/zh_cn/user_guides/useful_hooks.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
sanbuphy and others added 7 commits November 14, 2022 14:49
python tools/analysis_tools/analyze_logs.py plot_curve [--keys ${KEYS}] [--eval-interval ${EVALUATION_INTERVAL}] [--title ${TITLE}] [--legend ${LEGEND}] [--backend ${BACKEND}] [--style ${STYLE}] [--out ${OUT_FILE}]
```

![loss curve image](../../../resources/loss_curve.png)
Copy link
Collaborator

Choose a reason for hiding this comment

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

please check this image path whether is correct.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks for your modification, I have corrected all the mistakes. I think this image path is right because i can see it by Typora

docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
docs/zh_cn/user_guides/useful_tools.md Outdated Show resolved Hide resolved
sanbuphy and others added 4 commits November 21, 2022 12:42
Co-authored-by: BigDong <yudongwang1226@gmail.com>
Co-authored-by: BigDong <yudongwang1226@gmail.com>
Co-authored-by: BigDong <yudongwang1226@gmail.com>
@ZwwWayne ZwwWayne merged commit c455823 into open-mmlab:dev-3.x Nov 23, 2022
@sanbuphy sanbuphy deleted the docsbranch branch November 23, 2022 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants