Skip to content
Snippets Groups Projects

Implement CUDA Helmholtz sum-factorization kernels

All threads resolved!

This is a preliminary implementation of the Helmholtz operator. The Helmholtz operator is build by using a combination of BwdTrans, PhysDeriv, IProductWRTBase, and IProductWRTDerivBas operators. Both a StdMat and a CUDA versions have been implemented. Currently, unity diffusion coefficient is assumed along with lambda = 1. However, provision for non-unity, but constant, diffusion coefficient and a non-unity lambda parameter have been-made. The various sub-operators required to perform the Helmholtz operator are instantiated in the constructor.

Edited by Jacques Xing

Merge request reports

Approval is optional

Merged by Chris CantwellChris Cantwell 1 year ago (Sep 13, 2023 8:55am UTC)

Merge details

  • Changes merged into master with 2f79a27b (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Jacques Xing added 1 commit

    added 1 commit

    • 442ed4c3 - Add provision for non-unity lambda

    Compare with previous version

  • Jacques Xing added 1 commit

    added 1 commit

    Compare with previous version

  • Jacques Xing added 1 commit

    added 1 commit

    Compare with previous version

  • Jacques Xing added 1 commit

    added 1 commit

    • 9afd28f4 - Add provision for non-unity diffusion coefficient

    Compare with previous version

  • Jacques Xing resolved all threads

    resolved all threads

  • Jacques Xing changed the description

    changed the description

  • Jacques Xing changed the description

    changed the description

  • Jacques Xing added 3 commits

    added 3 commits

    • 9afd28f4...d05e90c5 - 2 commits from branch nektar:master
    • 3c4eeec0 - Merge remote-tracking branch 'upstream/master' into helmholtz_sum_fac_cuda_kernels

    Compare with previous version

  • Jacques Xing added 7 commits

    added 7 commits

    • 3c4eeec0...4360038b - 6 commits from branch nektar:master
    • 6388487b - Merge remote-tracking branch 'upstream/master' into helmholtz_sum_fac_cuda_kernels

    Compare with previous version

  • This looks reasonable to me, so will go ahead and merge. We could obviously do with some tests adding to ensure correctness, but probably best we sort them out for the constituent operators first.

  • merged

  • Chris Cantwell mentioned in commit 2f79a27b

    mentioned in commit 2f79a27b

  • Please register or sign in to reply
    Loading