Skip to content

fix bug of aparam size, should be nlocal_real#1664

Merged
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
amcadmus:fix-1662
Apr 28, 2022
Merged

fix bug of aparam size, should be nlocal_real#1664
wanghan-iapcm merged 1 commit intodeepmodeling:develfrom
amcadmus:fix-1662

Conversation

@amcadmus
Copy link
Member

@amcadmus amcadmus commented Apr 28, 2022

This PR fixes #1662

@amcadmus amcadmus requested a review from njzjz April 28, 2022 02:01
@codecov-commenter
Copy link

Codecov Report

Merging #1664 (d03a510) into devel (de7ba72) will decrease coverage by 11.86%.
The diff coverage is n/a.

❗ Current head d03a510 differs from pull request most recent head 62b886e. Consider uploading reports for the commit 62b886e to get more accurate results

@@             Coverage Diff             @@
##            devel    #1664       +/-   ##
===========================================
- Coverage   76.15%   64.28%   -11.87%     
===========================================
  Files          94        5       -89     
  Lines        7850       14     -7836     
===========================================
- Hits         5978        9     -5969     
+ Misses       1872        5     -1867     
Impacted Files Coverage Δ
deepmd/fit/__init__.py
source/op/_prod_force_se_a_grad.py
deepmd/entrypoints/transfer.py
deepmd/model/model.py
deepmd/__main__.py
deepmd/train/trainer.py
deepmd/__init__.py
deepmd/infer/deep_eval.py
deepmd/loss/ener.py
deepmd/fit/polar.py
... and 79 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 de7ba72...62b886e. Read the comment docs.

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.

[BUG] inconsistent dim of aparam when running MD with aparam-dependent-DP

4 participants