Skip to content

[Feature] Python bridge integration to AGCM#1205

Merged
mathomp4 merged 2 commits intodevelopfrom
fdeconinck/feat/python_bridge_init
Mar 12, 2026
Merged

[Feature] Python bridge integration to AGCM#1205
mathomp4 merged 2 commits intodevelopfrom
fdeconinck/feat/python_bridge_init

Conversation

@FlorianDeconinck
Copy link
Copy Markdown
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
Copy Markdown
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

@mathomp4 mathomp4 merged commit 7856dc3 into develop Mar 12, 2026
7 of 9 checks passed
@mathomp4 mathomp4 deleted the fdeconinck/feat/python_bridge_init branch March 12, 2026 19:17
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