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] Add a document about setting interval #959

Closed
wants to merge 18 commits into from

Conversation

YuetianW
Copy link
Contributor

Add a document about Setting the Frequency of Logging, Weight Saving, and Validation.

Add a document about Setting the Frequency of Logging, Weight Saving, and Validation.
@CLAassistant
Copy link

CLAassistant commented Feb 24, 2023

CLA assistant check
All committers have signed the CLA.

@YuetianW YuetianW changed the title Add a document about setting interval. [Docs]Add a document about setting interval. Feb 24, 2023
Fixed a small bug in the hyperlink.
Fixed the wrong hyperlink
@YuetianW
Copy link
Contributor Author

YuetianW commented Feb 25, 2023 via email

@zhouzaida
Copy link
Member

Hello! Thank you for your reply! It would be my pleasure if my PR could be merged. I hope that mmengine will be perfected step by step under the joint efforts of everyone!

Hi, thanks for your contribution. Let us make it better. We will review this PR ASAP.

@YuetianW
Copy link
Contributor Author

This PR link issue #962 .

@zhouzaida
Copy link
Member

Hi, the lint failed. You can refer to CONTRIBUTING.md#2-configure-pre-commit to fix it.

@YuetianW
Copy link
Contributor Author

YuetianW commented Feb 26, 2023

Hi, the lint failed. You can refer to CONTRIBUTING.md#2-configure-pre-commit to fix it.

Thanks for your reminder, it has been fixed by pre-commit.

@zhouzaida
Copy link
Member

Hi, please also refer to #917 to update the file en/zh_cn/index.rst.

@zhouzaida zhouzaida changed the title [Docs]Add a document about setting interval. [Docs] Add a document about setting interval Feb 26, 2023
YuetianW and others added 13 commits February 27, 2023 09:23
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Co-authored-by: Zaida Zhou <58739961+zhouzaida@users.noreply.github.com>
Added: common_usage/set_interval.md
@YuetianW
Copy link
Contributor Author

Hi, please also refer to #917 to update the file en/zh_cn/index.rst.

en/index.rst is updated in the latest commit!

@YuetianW YuetianW deleted the branch open-mmlab:main February 27, 2023 07:38
@YuetianW YuetianW closed this Feb 27, 2023
@YuetianW YuetianW deleted the main branch February 27, 2023 07:38
@YuetianW YuetianW restored the main branch February 27, 2023 07:38
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.

None yet

4 participants