Skip to content

fix restarting from compressed training with type embedding#2996

Merged
wanghan-iapcm merged 4 commits into
deepmodeling:develfrom
njzjz:fix-restart-compress-training-type-emd
Nov 20, 2023
Merged

fix restarting from compressed training with type embedding#2996
wanghan-iapcm merged 4 commits into
deepmodeling:develfrom
njzjz:fix-restart-compress-training-type-emd

Conversation

@njzjz

@njzjz njzjz commented Nov 16, 2023

Copy link
Copy Markdown
Member

Fix #2989.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@njzjz njzjz marked this pull request as ready for review November 16, 2023 23:05
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov

codecov Bot commented Nov 16, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (55d678d) 75.26% compared to head (8a73399) 75.62%.
Report is 1 commits behind head on devel.

Files Patch % Lines
deepmd/model/dos.py 0.00% 1 Missing ⚠️
deepmd/model/multi.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2996      +/-   ##
==========================================
+ Coverage   75.26%   75.62%   +0.36%     
==========================================
  Files         245      245              
  Lines       24670    24677       +7     
  Branches     1593     1593              
==========================================
+ Hits        18567    18663      +96     
+ Misses       5167     5076      -91     
- Partials      936      938       +2     

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

@njzjz njzjz requested review from nahso and wanghan-iapcm November 17, 2023 22:32
@nahso

nahso commented Nov 18, 2023

Copy link
Copy Markdown
Collaborator

This bug was not detected by unit tests? Maybe we should add correcponding unit tests as well.

@njzjz

njzjz commented Nov 18, 2023

Copy link
Copy Markdown
Member Author

This bug was not detected by unit tests? Maybe we should add correcponding unit tests as well.

Good point.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@wanghan-iapcm wanghan-iapcm merged commit a342e94 into deepmodeling:devel Nov 20, 2023
@njzjz njzjz added the bug label Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Restarting from compressed training with type embedding throws errors

3 participants