Skip to content

fix model conversion of 0.12#1941

Merged
wanghan-iapcm merged 2 commits into
deepmodeling:develfrom
njzjz:convert-0.12
Sep 23, 2022
Merged

fix model conversion of 0.12#1941
wanghan-iapcm merged 2 commits into
deepmodeling:develfrom
njzjz:convert-0.12

Conversation

@njzjz

@njzjz njzjz commented Sep 22, 2022

Copy link
Copy Markdown
Member

Fix #1934.

Add descrpt_attr/rcut, descrpt_attr/ntypes, fitting_attr/dfparam, model_type, o_energy, o_force, o_virial, o_atom_energy, o_atom_virial.

…`, and `model_type` to v0.12 models

fix deepmodeling#1934

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm September 22, 2022 08:51
@njzjz

njzjz commented Sep 22, 2022

Copy link
Copy Markdown
Member Author

I have tested it using LAMMPS.

@njzjz njzjz linked an issue Sep 22, 2022 that may be closed by this pull request
@codecov-commenter

codecov-commenter commented Sep 22, 2022

Copy link
Copy Markdown

Codecov Report

Base: 78.14% // Head: 78.12% // Decreases project coverage by -0.01% ⚠️

Coverage data is based on head (d71b9e1) compared to base (5aab6d7).
Patch coverage: 14.28% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #1941      +/-   ##
==========================================
- Coverage   78.14%   78.12%   -0.02%     
==========================================
  Files         118      118              
  Lines        9872     9875       +3     
==========================================
+ Hits         7714     7715       +1     
- Misses       2158     2160       +2     
Impacted Files Coverage Δ
deepmd/utils/convert.py 14.65% <14.28%> (+0.49%) ⬆️

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 f0973f0 into deepmodeling:devel Sep 23, 2022
@njzjz njzjz deleted the convert-0.12 branch October 30, 2022 04:28
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
Fix deepmodeling#1934.

Add `descrpt_attr/rcut`, `descrpt_attr/ntypes`, `fitting_attr/dfparam`,
`model_type`, `o_energy`, `o_force`, `o_virial`, `o_atom_energy`,
`o_atom_virial`.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
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.

[BUG] models converted from v0.12 failed to run LAMMPS

3 participants