Skip to content

support init_frz_model for hybrid descriptor#1112

Merged
amcadmus merged 2 commits intodeepmodeling:develfrom
njzjz:init_frz_model_hybrid
Sep 8, 2021
Merged

support init_frz_model for hybrid descriptor#1112
amcadmus merged 2 commits intodeepmodeling:develfrom
njzjz:init_frz_model_hybrid

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented Sep 7, 2021

Refactors some methods to implement it.
Also fixes some typos.

Refactors some methods to implement it.
Also fixes some typos.
@njzjz njzjz requested review from amcadmus and denghuilu September 7, 2021 22:04
@codecov-commenter
Copy link

codecov-commenter commented Sep 7, 2021

Codecov Report

Merging #1112 (7e48f71) into devel (2a71494) will decrease coverage by 0.14%.
The diff coverage is 33.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##            devel    #1112      +/-   ##
==========================================
- Coverage   75.61%   75.46%   -0.15%     
==========================================
  Files          88       88              
  Lines        7016     7036      +20     
==========================================
+ Hits         5305     5310       +5     
- Misses       1711     1726      +15     
Impacted Files Coverage Δ
deepmd/model/ener.py 91.74% <0.00%> (-0.85%) ⬇️
deepmd/train/trainer.py 72.01% <0.00%> (ø)
deepmd/model/tensor.py 88.46% <20.00%> (-0.86%) ⬇️
deepmd/descriptor/hybrid.py 36.84% <21.42%> (-3.49%) ⬇️
deepmd/descriptor/descriptor.py 75.00% <33.33%> (-6.49%) ⬇️
deepmd/utils/graph.py 74.11% <42.85%> (ø)
deepmd/descriptor/se_a.py 94.40% <75.00%> (+0.30%) ⬆️

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 2a71494...7e48f71. Read the comment docs.

Co-authored-by: Denghui Lu <ludenghui.cs@gmail.com>
@amcadmus amcadmus requested a review from denghuilu September 8, 2021 12:01
@amcadmus amcadmus merged commit 97be2f5 into deepmodeling:devel Sep 8, 2021
@njzjz njzjz deleted the init_frz_model_hybrid branch September 8, 2021 18:37
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.

4 participants