Skip to content

Conversation

@wanghan-iapcm
Copy link
Collaborator

No description provided.

Signed-off-by: Han Wang <92130845+wanghan-iapcm@users.noreply.github.com>
@wanghan-iapcm wanghan-iapcm changed the title All mixed type fix: replace all distinguished types by mixed types. Feb 18, 2024
@codecov
Copy link

codecov bot commented Feb 18, 2024

Codecov Report

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

Comparison is base (5600e45) 74.99% compared to head (b77cb3f) 75.00%.
Report is 1 commits behind head on devel.

Files Patch % Lines
deepmd/pt/model/task/fitting.py 66.66% 3 Missing ⚠️
deepmd/dpmodel/atomic_model/linear_atomic_model.py 66.66% 1 Missing ⚠️
deepmd/pt/model/descriptor/se_atten.py 75.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #3289      +/-   ##
==========================================
+ Coverage   74.99%   75.00%   +0.01%     
==========================================
  Files         392      392              
  Lines       33759    33757       -2     
  Branches     1604     1604              
==========================================
+ Hits        25319    25321       +2     
+ Misses       7579     7575       -4     
  Partials      861      861              

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

@wanghan-iapcm wanghan-iapcm added this pull request to the merge queue Feb 19, 2024
Merged via the queue into deepmodeling:devel with commit 9f6ff1e Feb 19, 2024
@wanghan-iapcm wanghan-iapcm deleted the all-mixed-type branch February 19, 2024 07:18
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Feb 19, 2024
It was broken by deepmodeling#3289, which replaced distinguished types by mixed types but didn't change the input.

Add tests for two types.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
github-merge-queue bot pushed a commit that referenced this pull request Feb 20, 2024
It was broken by #3289, which replaced distinguished types by mixed
types but didn't change the input.

Add tests for two types.

---------

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.

4 participants