Skip to content

remove warnings of batch size for mixed systems training#2470

Merged
wanghan-iapcm merged 1 commit into
deepmodeling:develfrom
njzjz:warn-batch-size-mixed-systems
May 19, 2023
Merged

remove warnings of batch size for mixed systems training#2470
wanghan-iapcm merged 1 commit into
deepmodeling:develfrom
njzjz:warn-batch-size-mixed-systems

Conversation

@njzjz

@njzjz njzjz commented Apr 22, 2023

Copy link
Copy Markdown
Member

Here for mixed systems, the system batch size is just a placeholder.

Here the system batch size is just a placeholder

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz changed the title remove warnings of batch size for mixed systems remove warnings of batch size for mixed systems training Apr 22, 2023
@codecov

codecov Bot commented Apr 22, 2023

Copy link
Copy Markdown

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (edb4e69) 74.38% compared to head (1a9e379) 74.38%.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2470   +/-   ##
=======================================
  Coverage   74.38%   74.38%           
=======================================
  Files         227      227           
  Lines       23401    23401           
  Branches     1673     1673           
=======================================
  Hits        17408    17408           
  Misses       4896     4896           
  Partials     1097     1097           
Impacted Files Coverage Δ
deepmd/utils/data_system.py 84.34% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wanghan-iapcm wanghan-iapcm requested a review from iProzd April 23, 2023 01:11
@wanghan-iapcm wanghan-iapcm merged commit 2c2feb1 into deepmodeling:devel May 19, 2023
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