Skip to content

Update parallel-training.md#1156

Merged
amcadmus merged 5 commits intodeepmodeling:develfrom
chazeon:chazeon-patch-distributed-training-doc
Sep 17, 2021
Merged

Update parallel-training.md#1156
amcadmus merged 5 commits intodeepmodeling:develfrom
chazeon:chazeon-patch-distributed-training-doc

Conversation

@chazeon
Copy link
Contributor

@chazeon chazeon commented Sep 15, 2021

Add additional information for parallel training documentation.

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

Codecov Report

Merging #1156 (cfe7a38) into devel (0a46641) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            devel    #1156   +/-   ##
=======================================
  Coverage   75.66%   75.66%           
=======================================
  Files          89       89           
  Lines        7048     7048           
=======================================
  Hits         5333     5333           
  Misses       1715     1715           

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 0a46641...cfe7a38. Read the comment docs.

@amcadmus
Copy link
Member

amcadmus commented Sep 16, 2021

The environment setup part has been moved to the installation doc #1147 do you mind moving the check-build part of your PR to the installation?

@chazeon chazeon force-pushed the chazeon-patch-distributed-training-doc branch from a2b1866 to d11c6b6 Compare September 16, 2021 03:24
@chazeon chazeon force-pushed the chazeon-patch-distributed-training-doc branch from d11c6b6 to 8ae1125 Compare September 16, 2021 03:25
Trying to be more specific in the parallel training description
@amcadmus amcadmus merged commit 57dd29b into deepmodeling:devel Sep 17, 2021
njzjz pushed a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
previous natoms only count #atom of the first element, gives wrong
results or -inf in multiple-element cases

Signed-off-by: Wanrun Jiang <58099845+Vibsteamer@users.noreply.github.com>
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.

5 participants