Skip to content

fix: remove model_def_script from AtomicModel#3449

Merged
njzjz merged 3 commits intodeepmodeling:develfrom
njzjz:fix-model-def-script
Mar 12, 2024
Merged

fix: remove model_def_script from AtomicModel#3449
njzjz merged 3 commits intodeepmodeling:develfrom
njzjz:fix-model-def-script

Conversation

@njzjz
Copy link
Copy Markdown
Member

@njzjz njzjz commented Mar 12, 2024

After #3438, model_def_script is no more saved in AtomicModel.

njzjz added 2 commits March 11, 2024 19:58
After deepmodeling#3438, `model_def_script` is no more saved in AtomicModel.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz requested a review from wanghan-iapcm March 12, 2024 00:01
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2024

Codecov Report

Attention: Patch coverage is 88.88889% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 77.29%. Comparing base (bc35ac9) to head (7094868).

Files Patch % Lines
deepmd/dpmodel/model/base_model.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #3449   +/-   ##
=======================================
  Coverage   77.29%   77.29%           
=======================================
  Files         427      427           
  Lines       36913    36910    -3     
  Branches     1621     1621           
=======================================
- Hits        28531    28529    -2     
+ Misses       7514     7513    -1     
  Partials      868      868           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz added this pull request to the merge queue Mar 12, 2024
Merged via the queue into deepmodeling:devel with commit 9bcae14 Mar 12, 2024
@njzjz njzjz deleted the fix-model-def-script branch March 12, 2024 06:23
github-merge-queue Bot pushed a commit that referenced this pull request Mar 12, 2024
This PR is based on #3449, as the test needs #3449 to pass.

Add a consistent `frozen` model in pt. Both TF and PT now support using
models in any format.

---------

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz mentioned this pull request Apr 2, 2024
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.

2 participants