Skip to content

Add KOKKOS version of dihedral style nharmonic#4933

Merged
akohlmey merged 7 commits into
lammps:developfrom
akohlmey:dihedral-nharmonic-kokkos
Mar 30, 2026
Merged

Add KOKKOS version of dihedral style nharmonic#4933
akohlmey merged 7 commits into
lammps:developfrom
akohlmey:dihedral-nharmonic-kokkos

Conversation

@akohlmey

@akohlmey akohlmey commented Mar 29, 2026

Copy link
Copy Markdown
Member

Summary

This ports the dihedral style nharmonic to the KOKKOS package.

Related Issue(s)

Implements and thus closes #4931

Author(s)

Axel Kohlmeyer, Temple U

Licensing

By submitting this pull request, I agree, that my contribution will be included in LAMMPS and redistributed under either the GNU General Public License version 2 (GPL v2) or the GNU Lesser General Public License version 2.1 (LGPL v2.1).

Artificial Intelligence (AI) Tools Usage

The porting was done almost completely by GitHub Copilot using Claude Opus 4.6.
Axel Kohlmeyer only fixed a couple of bugs including the restart bug that was
just today discovered and corrected in dihedral style fourier in PR #4929.

Backward Compatibility

Binary restarts for dihedral nharmonic are not compatible with older versions.

Post Submission Checklist

  • The feature or features in this pull request is complete
  • Licensing information is complete
  • Corresponding author information is complete
  • The source code follows the LAMMPS formatting guidelines
  • Suitable new documentation files and/or updates to the existing docs are included
  • The added/updated documentation is integrated and tested with the documentation build system
  • The feature has been verified to work with the CMake based build system

Copilot AI and others added 7 commits March 28, 2026 21:48
Create dihedral_nharmonic_kokkos.h and .cpp in src/KOKKOS/ following
the pattern from dihedral_multi_harmonic_kokkos and dihedral_fourier_kokkos.
Uses 2D Kokkos views for variable-length coefficient arrays and 1D
int views for per-type term counts. Update documentation to list the
new nharmonic/kk accelerated variant.

Agent-Logs-Url: https://github.com/akohlmey/lammps/sessions/5d795956-0b37-4808-837f-9a840867ed96

Co-authored-by: akohlmey <289251+akohlmey@users.noreply.github.com>
@akohlmey akohlmey moved this to Ready for Review or Merge in LAMMPS Pull Requests Mar 29, 2026
@akohlmey
akohlmey requested a review from ndtrung81 March 29, 2026 02:26
@akohlmey
akohlmey marked this pull request as ready for review March 29, 2026 02:46
@akohlmey
akohlmey requested a review from stanmoore1 as a code owner March 29, 2026 02:46
@akohlmey
akohlmey requested review from athomps and sjplimp March 29, 2026 02:46
@akohlmey akohlmey assigned akohlmey and unassigned stanmoore1 Mar 29, 2026

@athomps athomps left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I approve

@akohlmey
akohlmey merged commit 98971b0 into lammps:develop Mar 30, 2026
14 checks passed
@github-project-automation github-project-automation Bot moved this from Ready for Review or Merge to Done in LAMMPS Pull Requests Mar 30, 2026
@akohlmey
akohlmey deleted the dihedral-nharmonic-kokkos branch March 30, 2026 14:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Development

Successfully merging this pull request may close these issues.

Add KOKKOS support for dihedral_style nharmonic (nharmonic/kk)

5 participants