Skip to content

Add model deviation of virial#594

Merged
amcadmus merged 5 commits intodeepmodeling:develfrom
amcadmus:devel
May 8, 2021
Merged

Add model deviation of virial#594
amcadmus merged 5 commits intodeepmodeling:develfrom
amcadmus:devel

Conversation

@amcadmus
Copy link
Member

@amcadmus amcadmus commented May 7, 2021

  • add model deviation and relative model deviation of virial
  • refact the computation of std and relative std
  • UT covering the case of relative std

@amcadmus amcadmus requested a review from njzjz May 7, 2021 06:54
@codecov-commenter
Copy link

codecov-commenter commented May 7, 2021

Codecov Report

Merging #594 (6218faf) into devel (850169f) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##            devel     #594   +/-   ##
=======================================
  Coverage   62.82%   62.82%           
=======================================
  Files          75       75           
  Lines        6333     6333           
=======================================
  Hits         3979     3979           
  Misses       2354     2354           

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 850169f...6218faf. Read the comment docs.

Copy link
Member

@njzjz njzjz left a comment

Choose a reason for hiding this comment

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

LGTM

@amcadmus amcadmus merged commit 75d7ca1 into deepmodeling:devel May 8, 2021
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Aug 30, 2021
amcadmus pushed a commit that referenced this pull request Aug 31, 2021
Fix #1068.
This bug is produced by #594.
gzq942560379 pushed a commit to HPC-AI-Team/deepmd-kit that referenced this pull request Sep 1, 2021
njzjz pushed a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
Change-Id: I12cc467e5956a5deb29ac4bd43719db1c0ef8942

Co-authored-by: Zhengju Sha <jenny@bytedance.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.

3 participants