Skip to content

add ut for model compression#586

Merged
amcadmus merged 18 commits intodeepmodeling:develfrom
denghuilu:devel
May 9, 2021
Merged

add ut for model compression#586
amcadmus merged 18 commits intodeepmodeling:develfrom
denghuilu:devel

Conversation

@denghuilu
Copy link
Member

changes

  • add ut for model compression
  • fix bugs of model compression

@njzjz njzjz linked an issue May 4, 2021 that may be closed by this pull request
@denghuilu denghuilu closed this May 6, 2021
@denghuilu denghuilu reopened this May 6, 2021
@codecov-commenter
Copy link

codecov-commenter commented May 9, 2021

Codecov Report

Merging #586 (f06e8ab) into devel (850169f) will increase coverage by 11.74%.
The diff coverage is 84.21%.

Impacted file tree graph

@@             Coverage Diff             @@
##            devel     #586       +/-   ##
===========================================
+ Coverage   62.82%   74.57%   +11.74%     
===========================================
  Files          75       75               
  Lines        6333     6348       +15     
===========================================
+ Hits         3979     4734      +755     
+ Misses       2354     1614      -740     
Impacted Files Coverage Δ
deepmd/fit/ener.py 94.32% <ø> (+1.03%) ⬆️
deepmd/utils/tabulate.py 88.69% <50.00%> (+72.83%) ⬆️
deepmd/descriptor/se_a.py 97.11% <100.00%> (+6.53%) ⬆️
deepmd/entrypoints/compress.py 97.22% <100.00%> (+62.93%) ⬆️
deepmd/utils/argcheck.py 88.02% <100.00%> (+71.66%) ⬆️
deepmd/utils/data.py 91.32% <0.00%> (+0.68%) ⬆️
deepmd/common.py 83.08% <0.00%> (+2.94%) ⬆️
deepmd/utils/data_system.py 71.28% <0.00%> (+3.21%) ⬆️
deepmd/utils/compat.py 83.33% <0.00%> (+5.55%) ⬆️
... and 15 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 850169f...f06e8ab. Read the comment docs.

@amcadmus amcadmus merged commit 33c0c0f into deepmodeling:devel May 9, 2021
This was referenced Jun 10, 2021
gzq942560379 pushed a commit to HPC-AI-Team/deepmd-kit that referenced this pull request Sep 1, 2021
* add ut for model compression

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* fix bug of UT

* adjust code structure model compression
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.

Model compression error: descriptor type must be se_a!

3 participants