Closed hfp closed 1 year ago
@hfp Is this a "replacement" of the #647 ? Please, do not merge for the moment, I would like to review #647 and test on LUMI before any merge...
@hfp Is this a "replacement" of the #647 ? Please, do not merge for the moment, I would like to review #647 and test on LUMI before any merge...
No! Please do not merge. This is simply to check if my change would solve some of the issues in #647 if @gsitaram decides to rebase/merge the upstream master. Also, I have one more change on top of #647 but I will share it as a comment.
Base: 66.3% // Head: 66.2% // Decreases project coverage by -0.2%
:warning:
Coverage data is based on head (
24e77ee
) compared to base (82f2b64
). Patch coverage: 4.7% of modified lines in pull request are covered.
:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.
All tests passed except "Testing on Linux / build-on-rocm (OPENMP=ON) (pull_request)".