Skip to content

fix build_type_exclude_mask when nloc != nall#2505

Merged
wanghan-iapcm merged 2 commits intodeepmodeling:develfrom
njzjz:fix-dprc-mask-nloc-nall
May 3, 2023
Merged

fix build_type_exclude_mask when nloc != nall#2505
wanghan-iapcm merged 2 commits intodeepmodeling:develfrom
njzjz:fix-dprc-mask-nloc-nall

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented May 3, 2023

No description provided.

@njzjz njzjz requested a review from wanghan-iapcm May 3, 2023 21:09
@github-actions github-actions bot added the Python label May 3, 2023
@codecov
Copy link

codecov bot commented May 3, 2023

Codecov Report

Patch coverage: 100.00% and no project coverage change.

Comparison is base (b13b26c) 74.40% compared to head (67947da) 74.40%.

Additional details and impacted files
@@           Coverage Diff           @@
##            devel    #2505   +/-   ##
=======================================
  Coverage   74.40%   74.40%           
=======================================
  Files         227      227           
  Lines       23428    23430    +2     
  Branches     1666     1680   +14     
=======================================
+ Hits        17431    17434    +3     
  Misses       4890     4890           
+ Partials     1107     1106    -1     
Impacted Files Coverage Δ
deepmd/descriptor/se_atten.py 94.46% <ø> (ø)
deepmd/descriptor/se_a.py 95.64% <100.00%> (+0.01%) ⬆️
deepmd/descriptor/se_r.py 96.50% <100.00%> (+0.01%) ⬆️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@wanghan-iapcm wanghan-iapcm merged commit 66b5f6b into deepmodeling:devel May 3, 2023
njzjz added a commit to njzjz/deepmd-kit that referenced this pull request Dec 5, 2023
See also deepmodeling#2390 and deepmodeling#2505...

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
wanghan-iapcm pushed a commit that referenced this pull request Dec 7, 2023
See also #2390 and #2505...

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
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