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

[4.4.00] Cherry-pick PRs 2281 and 2297 #2301

Conversation

ndellingwood
Copy link
Contributor

jgfouca added 2 commits August 7, 2024 12:46
* Some cleanup and refactoring

* First round of cleanup complete

* Fix a couple warnings

* formatting

* Cleanup progress

* Fixes

* Remove Upper/Lower TriLvlSchedTP2SolverFunctors

* Remove Upper/Lower single block functors

* Remove unused TriLvlSchedTP1SingleBlockFunctorDiagValues and merge upper/lower tri_solve_cg

* Merge two big upper/lower branch of tri_solve_chain

* Merge upper/lower tri_solve_streams

* progres

* progress

* Progress, test added

* progress and fixes

* prog

* prog but broken

* Prog with debug prints

* Remove extreme debug printing

* works

* all working

* Remove test mangling

* Switch over block spiluk precond test to use new block sptrsv

* More test cleanup

* Fixes for GPU warnings

* Conflicts resolved, still work needed

* builds

* Formatting

* Update work to latest format style

* Remove unused functions. Remove prints. Add barriers

* Minor fixes

* lset is not used

* Fix for clang

* formatting

* New impl approach

* Unset macro

* There's no reason to limit team-policy alg to MAX_VEC_SIZE

* formatting

* Add missing kokkos-inline-funcs

* Fix warnings

* format
@ndellingwood ndellingwood requested review from jgfouca and lucbv August 7, 2024 18:49
@ndellingwood
Copy link
Contributor Author

I don't understand why three of the jobs were cancelled , maybe adding a review triggered something in AT2? @lucbv ?

@ndellingwood
Copy link
Contributor Author

The jobs that were cancelled after the review submission had matching configuration jobs with extra label (pull_request_review) launch (the cancelled jobs labeled as pull_request). I suspect we don't need to relaunch the canceled jobs, they would just duplicate the tests

@ndellingwood
Copy link
Contributor Author

Cancelled jobs were superceded by those with the modified pull_request_review label suffix, ready for merge

@ndellingwood ndellingwood merged commit 418e2fa into kokkos:release-candidate-4.4.00 Aug 7, 2024
9 of 12 checks passed
@ndellingwood ndellingwood deleted the rc-4.4.00-cp-2281-2297 branch August 7, 2024 22:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants