Skip to content

[NDSL | GFDL1M] Interface performance engineering#1186

Merged
FlorianDeconinck merged 12 commits intoGEOS-ESM:dsl/developfrom
FlorianDeconinck:dsl/refactor/GFDL1M_GPU
Jan 6, 2026
Merged

[NDSL | GFDL1M] Interface performance engineering#1186
FlorianDeconinck merged 12 commits intoGEOS-ESM:dsl/developfrom
FlorianDeconinck:dsl/refactor/GFDL1M_GPU

Conversation

@FlorianDeconinck
Copy link
Collaborator

We rework the Fortran/Python interface to expose three mode of computation: CPU_COPY, CPU_MAP and CPU_GPU. Those modes are resolved at init time looking at the backend called for NDSL.

The memory mapping brings the cost of interface to a negligible amount and hashing scales it with core count properly.

Also:

  • some light fixes to interface code that was abusing of the "extra point" in common backend
  • some fixes to the translate tests & initialization of code

⚠️ This code is dependent on soon-to-be-merged updates to NDSL ⚠️

@FlorianDeconinck FlorianDeconinck requested a review from a team as a code owner December 30, 2025 17:59
@FlorianDeconinck FlorianDeconinck added 0 diff structural Structural changes to repository that are zero-diff 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) and removed 0 diff structural Structural changes to repository that are zero-diff labels Dec 30, 2025
@FlorianDeconinck
Copy link
Collaborator Author

Blocked behind: NOAA-GFDL/NDSL#348

… as persistent

Removing unsued transient reset after computations is done in stencil (unused)
@FlorianDeconinck FlorianDeconinck requested a review from a team as a code owner January 6, 2026 15:02
@FlorianDeconinck FlorianDeconinck removed request for a team January 6, 2026 16:57
@FlorianDeconinck FlorianDeconinck merged commit 50f63e8 into GEOS-ESM:dsl/develop Jan 6, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments