Skip to content
Snippets Groups Projects

Merge consolidation and transformation

Merged Matt Martineau requested to merge mmartin2/foam2csr:master into master

This has merged consolidation and transformation, which freed us from issues limiting the number of cores that could be efficiently selected. Note we still face a context overhead penalty due to the number of ranks interacting with the GPU in order to maximise performance.

There are API changes that will require minor adjustments to the calls in PETSC4FOAM (a couple of lines removed).

AmgXSolver::solve(psi, source, A) -> AmgXSolver::solve(nrows, psi_raw, source_raw, A)

AmgXSolver::setOperator(nrowsLocal, nrowsGlobal, nnz, A, nullptr) -> AmgXSolver::setOperator(nrowsLocal, nrowsGlobal, nnz, A)

Also there will be an additional call to set up the communicator for the system.

AmgXSolver::initialiseMatrixComms(A);

Happy to take guidance on how this should be changed to match your expectations of style etc.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Loading
  • Loading
Please register or sign in to reply
Loading