Skip to content

add se_a_type & unittest#572

Closed
yhliu918 wants to merge 2 commits intodeepmodeling:develfrom
yhliu918:devel
Closed

add se_a_type & unittest#572
yhliu918 wants to merge 2 commits intodeepmodeling:develfrom
yhliu918:devel

Conversation

@yhliu918
Copy link
Contributor

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Apr 27, 2021

Codecov Report

Merging #572 (d576ebb) into devel (a5cb130) will increase coverage by 0.74%.
The diff coverage is 74.08%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel     #572      +/-   ##
==========================================
+ Coverage   62.09%   62.84%   +0.74%     
==========================================
  Files          74       76       +2     
  Lines        6229     6693     +464     
==========================================
+ Hits         3868     4206     +338     
- Misses       2361     2487     +126     
Impacted Files Coverage Δ
deepmd/utils/argcheck.py 15.84% <5.55%> (-0.70%) ⬇️
deepmd/train/trainer.py 38.13% <25.00%> (-0.20%) ⬇️
deepmd/fit/ener.py 78.00% <55.00%> (-7.63%) ⬇️
deepmd/utils/network.py 72.26% <61.53%> (-24.03%) ⬇️
deepmd/descriptor/se_a_type.py 82.18% <82.18%> (ø)
deepmd/descriptor/__init__.py 100.00% <100.00%> (ø)
deepmd/model/ener.py 90.90% <100.00%> (+0.58%) ⬆️
deepmd/utils/type_embed.py 100.00% <100.00%> (ø)
deepmd/loss/tensor.py 32.53% <0.00%> (ø)
... and 2 more

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 a5cb130...d576ebb. Read the comment docs.

@amcadmus
Copy link
Member

Replaced by PR #581

@amcadmus amcadmus closed this Apr 30, 2021
njzjz pushed a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
…pmodeling#572)

Change-Id: Ia138ee7c31b6c41d9f41f5943affa9ebf8803c46

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