Skip to content

nvnmd: fix some warnings about matmul_flt_nvnmd#2430

Merged
wanghan-iapcm merged 50 commits intodeepmodeling:develfrom
LiuGroupHNU:devel
Mar 29, 2023
Merged

nvnmd: fix some warnings about matmul_flt_nvnmd#2430
wanghan-iapcm merged 50 commits intodeepmodeling:develfrom
LiuGroupHNU:devel

Conversation

@LiuGroupHNU
Copy link
Collaborator

No description provided.

@codecov
Copy link

codecov bot commented Mar 28, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (8568521) 75.19% compared to head (6aa8dfb) 75.19%.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2430      +/-   ##
==========================================
- Coverage   75.19%   75.19%   -0.01%     
==========================================
  Files         223      223              
  Lines       22586    22592       +6     
  Branches     1669     1673       +4     
==========================================
+ Hits        16984    16987       +3     
  Misses       4510     4510              
- Partials     1092     1095       +3     
Impacted Files Coverage Δ
source/op/matmul_flt_nvnmd.cc 83.13% <0.00%> (-3.89%) ⬇️

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ 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 requested a review from njzjz March 28, 2023 04:04
@wanghan-iapcm wanghan-iapcm merged commit c10cd6c into deepmodeling:devel Mar 29, 2023
nahso pushed a commit to HPC-AI-Team/deepmd-kit-se_atten-models that referenced this pull request Apr 12, 2023
Co-authored-by: LiuGroupHNU <liujie123@HNU>
Co-authored-by: MoPinghui <mopinghui1020@gmail.com>
Co-authored-by: Han Wang <92130845+wanghan-iapcm@users.noreply.github.com>
Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
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.

4 participants