Skip to content

[Feature] Python bridge integration to AGCM#1205

Open
FlorianDeconinck wants to merge 1 commit intodevelopfrom
fdeconinck/feat/python_bridge_init
Open

[Feature] Python bridge integration to AGCM#1205
FlorianDeconinck wants to merge 1 commit intodevelopfrom
fdeconinck/feat/python_bridge_init

Conversation

@FlorianDeconinck
Copy link
Collaborator

@FlorianDeconinck FlorianDeconinck commented Feb 6, 2026

⚠️ ⚠️ This is a sister PR to MAPL#4369 and should not be merged before ⚠️ ⚠️ Merged!

This requries a MAPL release (poke @mathomp4)

This PR spins the python bridge with AGCM local grid size.

This is mostly required because the Python-based MAPLPy.GetPointer requires to be given rank and dim sizes to transform the memory properly. We record the local grid size in order to allow the user to access them from within the Python.

@FlorianDeconinck FlorianDeconinck requested a review from a team as a code owner February 6, 2026 16:57
@FlorianDeconinck FlorianDeconinck added the 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) label Feb 6, 2026
@FlorianDeconinck FlorianDeconinck added the Contingent - DNA These changes are contingent on other PRs (DNA=do not approve) label Feb 6, 2026
@mathomp4 mathomp4 added 0 diff The changes in this pull request have verified to be zero-diff with the target branch. and removed 0 diff trivial The changes in this pull request are trivially zero-diff (documentation, build failure, &c.) labels Feb 6, 2026
@FlorianDeconinck FlorianDeconinck removed the Contingent - DNA These changes are contingent on other PRs (DNA=do not approve) label Feb 27, 2026
@mathomp4
Copy link
Member

⚠️ ⚠️ This is a sister PR to MAPL#4369 and should not be merged before ⚠️ ⚠️ Merged!

This requries a MAPL release (poke @mathomp4)

MAPL 2.67 has been released:

https://github.com/GEOS-ESM/MAPL/releases/tag/v2.67.0

and I've updated my "Update MAPL in GEOSgcm" PR to have it:

GEOS-ESM/GEOSgcm#1034

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

0 diff The changes in this pull request have verified to be zero-diff with the target branch.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants