Skip to content

reduce training steps in tests #1771

Merged
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:reduce_training_steps
Jun 19, 2022
Merged

reduce training steps in tests #1771
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
njzjz:reduce_training_steps

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Jun 19, 2022

From the perspective of unit tests, there is no difference between training 1 step and 100 steps.

From the perspective of unit tests, there is no differece between testing 1 step and 100 steps.
@njzjz njzjz requested a review from denghuilu June 19, 2022 00:56
@codecov-commenter
Copy link

codecov-commenter commented Jun 19, 2022

Codecov Report

Merging #1771 (b02d6ed) into devel (3270244) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##            devel    #1771   +/-   ##
=======================================
  Coverage   76.12%   76.12%           
=======================================
  Files          96       96           
  Lines        7950     7950           
=======================================
  Hits         6052     6052           
  Misses       1898     1898           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3270244...b02d6ed. Read the comment docs.

@wanghan-iapcm wanghan-iapcm merged commit 731cbbc into deepmodeling:devel Jun 19, 2022
@njzjz njzjz deleted the reduce_training_steps branch June 19, 2022 02:51
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
From the perspective of unit tests, there is no differece between testing 1 step and 100 steps.
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.

3 participants