Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Model compression checkpoint #1076

Merged
merged 3 commits into from
Sep 1, 2021

Conversation

denghuilu
Copy link
Member

This PR change the default model compression checkpoint path from '.' to 'model-compression'.

In the original implementation, the model compression process will overwrite the checkpoint in the current directory, which may cause damage to the checkpoint of the original model.

@denghuilu denghuilu requested review from njzjz and amcadmus August 31, 2021 06:45
@codecov-commenter
Copy link

codecov-commenter commented Aug 31, 2021

Codecov Report

Merging #1076 (4bb2a10) into devel (75160c7) will decrease coverage by 10.90%.
The diff coverage is n/a.

❗ Current head 4bb2a10 differs from pull request most recent head 701fbff. Consider uploading reports for the commit 701fbff to get more accurate results
Impacted file tree graph

@@             Coverage Diff             @@
##            devel    #1076       +/-   ##
===========================================
- Coverage   75.19%   64.28%   -10.91%     
===========================================
  Files          82        5       -77     
  Lines        6945       14     -6931     
===========================================
- Hits         5222        9     -5213     
+ Misses       1723        5     -1718     
Impacted Files Coverage Δ
deepmd/descriptor/hybrid.py
deepmd/model/ener.py
deepmd/__main__.py
deepmd/descriptor/__init__.py
source/op/_prod_force_se_r_grad.py
deepmd/infer/__init__.py
deepmd/utils/data_system.py
deepmd/utils/compat.py
deepmd/calculator.py
deepmd/fit/polar.py
... and 75 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 75160c7...701fbff. Read the comment docs.

@amcadmus amcadmus merged commit c716b9f into deepmodeling:devel Sep 1, 2021
njzjz pushed a commit to njzjz/deepmd-kit that referenced this pull request Sep 21, 2023
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