Skip to content

fix dp freeze for multi-task when atom_ener is set#2239

Merged
wanghan-iapcm merged 1 commit into
deepmodeling:develfrom
njzjz:fix-dp-freeze-multi-atom-ener
Jan 11, 2023
Merged

fix dp freeze for multi-task when atom_ener is set#2239
wanghan-iapcm merged 1 commit into
deepmodeling:develfrom
njzjz:fix-dp-freeze-multi-atom-ener

Conversation

@njzjz

@njzjz njzjz commented Jan 10, 2023

Copy link
Copy Markdown
Member

When atom_ener is set, there will be variables named layer_xx_suffix_1/MatMul (the automatic name when layer_xx_suffix/MatMul has been used).

@njzjz njzjz requested a review from iProzd January 10, 2023 22:56
@njzjz njzjz changed the title fix dp freeze for multi-task when atom_ener is set fix dp freeze for multi-task when atom_ener is set Jan 10, 2023
@codecov-commenter

codecov-commenter commented Jan 10, 2023

Copy link
Copy Markdown

Codecov Report

Base: 75.66% // Head: 75.66% // Increases project coverage by +0.00% 🎉

Coverage data is based on head (624d592) compared to base (6154494).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2239   +/-   ##
=======================================
  Coverage   75.66%   75.66%           
=======================================
  Files         204      204           
  Lines       20249    20249           
  Branches     1438     1438           
=======================================
+ Hits        15321    15322    +1     
  Misses       4021     4021           
+ Partials      907      906    -1     
Impacted Files Coverage Δ
deepmd/env.py 72.51% <ø> (ø)
source/lib/src/fmt_nlist.cc 79.51% <0.00%> (+1.20%) ⬆️

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 at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wanghan-iapcm wanghan-iapcm merged commit 30920ab into deepmodeling:devel Jan 11, 2023
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
…2239)

When `atom_ener` is set, there will be variables named
`layer_xx_suffix_1/MatMul` (the automatic name when
`layer_xx_suffix/MatMul` has been used).
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.

4 participants