Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

TridiagSolver: remove unused kernels #906

Merged
merged 1 commit into from
Jul 7, 2023

Conversation

albestro
Copy link
Collaborator

@albestro albestro commented Jun 14, 2023

After #904 (and #860) many kernels are not used anymore.

Keeping this as separated fork of #904 in order to make a bit more easier the review of its algorithmic part.

@albestro albestro force-pushed the alby/rank1-problem-dist branch 3 times, most recently from f2e614c to 49c63fe Compare June 19, 2023 08:01
@albestro albestro changed the base branch from alby/rank1-problem-dist to master June 22, 2023 16:01
@albestro albestro force-pushed the alby/cleanup_solver_unused_kernels branch from 8ffa877 to e3bad0e Compare July 7, 2023 12:30
@albestro albestro marked this pull request as ready for review July 7, 2023 12:32
@albestro
Copy link
Collaborator Author

albestro commented Jul 7, 2023

cscs-ci run

@codecov-commenter
Copy link

Codecov Report

Merging #906 (e3bad0e) into master (6fcb08d) will increase coverage by 3.65%.
The diff coverage is n/a.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##           master     #906      +/-   ##
==========================================
+ Coverage   91.21%   94.86%   +3.65%     
==========================================
  Files         136      122      -14     
  Lines        8488     7460    -1028     
  Branches     1119     1018     -101     
==========================================
- Hits         7742     7077     -665     
+ Misses        572      229     -343     
+ Partials      174      154      -20     
Impacted Files Coverage Δ
include/dlaf/eigensolver/tridiag_solver/kernels.h 100.00% <ø> (ø)
include/dlaf/eigensolver/tridiag_solver/merge.h 100.00% <ø> (ø)
src/eigensolver/tridiag_solver/kernels.cpp 100.00% <ø> (+68.57%) ⬆️

... and 34 files with indirect coverage changes

@albestro albestro requested a review from rasolca July 7, 2023 13:04
@rasolca rasolca merged commit 670830b into master Jul 7, 2023
@rasolca rasolca deleted the alby/cleanup_solver_unused_kernels branch July 7, 2023 13:16
github-actions bot pushed a commit that referenced this pull request Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants