Skip to content

[SCAFFOLD] AMReX shim layer for {zonal,meridional}_edge_thickness#28

Open
johnmauff wants to merge 24 commits into
TURBO-ESM:dev/turbo-debugfrom
johnmauff:continuity05
Open

[SCAFFOLD] AMReX shim layer for {zonal,meridional}_edge_thickness#28
johnmauff wants to merge 24 commits into
TURBO-ESM:dev/turbo-debugfrom
johnmauff:continuity05

Conversation

@johnmauff
Copy link
Copy Markdown

This PR adds a Fortran shim layer for the {zonal, meridional}_edge_thickness functions. Additionally, the function set_continuity_box was added, which provides a function that creates a box iterator for the entire continuity equation solver

johnmauff and others added 17 commits April 11, 2026 17:36
…KNESS_MODE)

Rename original implementation to zonal_edge_thickness_fortran, adapting it
to take RealArray_t containers and explicit scalar args (h_min, upwind_1st,
monotonic, simple_2nd) in place of the G/GV/CS structs. Add new shim
zonal_edge_thickness with the same public signature: wraps raw arrays into
containers, dispatches via ZONAL_EDGE_THICKNESS_MODE={FORTRAN,CAPTURE,AMREX}.
Capture mode records inputs and outputs to capture/zonal_edge_thickness.{bin,meta}.
Add turbotmp_zonal_edge_thickness_bridge bind(C) interface for the C++/AMReX side.
No callers needed updating (5 call sites in MOM_continuity_PPM.F90 unchanged).

Co-authored-by: Claude <noreply@anthropic.com>
…idge skill

Co-authored-by: Claude <noreply@anthropic.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

Comment thread src/core/MOM_continuity_PPM.F90
Comment thread src/core/MOM_continuity_PPM.F90
Comment thread src/core/MOM_continuity_PPM.F90
Comment thread src/core/MOM_continuity.F90
Comment thread src/core/MOM_continuity_PPM.F90 Outdated
Comment thread src/core/MOM_continuity_PPM.F90 Outdated
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.

3 participants