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

support dp convert-from 1.0 #1597

Merged
merged 1 commit into from
Mar 25, 2022
Merged

Conversation

njzjz
Copy link
Member

@njzjz njzjz commented Mar 25, 2022

#1583
The commit also contains some common docstring fixes.

deepmodeling#1583
The commit also contains some common docstring fixes.
@njzjz
Copy link
Member Author

njzjz commented Mar 25, 2022

Provide a v1.0 model here for testing:
frozen_model.pb.txt

@codecov-commenter
Copy link

codecov-commenter commented Mar 25, 2022

Codecov Report

Merging #1597 (e4f4c0a) into devel (5d79471) will decrease coverage by 0.18%.
The diff coverage is 19.23%.

@@            Coverage Diff             @@
##            devel    #1597      +/-   ##
==========================================
- Coverage   76.75%   76.56%   -0.19%     
==========================================
  Files          93       93              
  Lines        7794     7817      +23     
==========================================
+ Hits         5982     5985       +3     
- Misses       1812     1832      +20     
Impacted Files Coverage Δ
deepmd/entrypoints/main.py 87.61% <ø> (ø)
deepmd/utils/plugin.py 84.37% <ø> (ø)
deepmd/utils/convert.py 15.46% <18.18%> (-0.33%) ⬇️
deepmd/entrypoints/convert.py 18.18% <25.00%> (-4.05%) ⬇️

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 5d79471...e4f4c0a. Read the comment docs.

@wanghan-iapcm wanghan-iapcm merged commit 27c297c into deepmodeling:devel Mar 25, 2022
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Oct 12, 2022
Strange, I added duplicated `convert_13_to_21` in deepmodeling#1597.

Signed-off-by: Jinzhe Zeng <[email protected]>
wanghan-iapcm pushed a commit that referenced this pull request Oct 13, 2022
Strange, I added duplicated `convert_13_to_21` in #1597.

Signed-off-by: Jinzhe Zeng <[email protected]>
mingzhong15 pushed a commit to mingzhong15/deepmd-kit that referenced this pull request Jan 15, 2023
Strange, I added duplicated `convert_13_to_21` in deepmodeling#1597.

Signed-off-by: Jinzhe Zeng <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[feature request] dp convert supports models trained with v0.12, v1.0 and v1.1
3 participants