Skip to content

Integrate Python code that computes temperature increments via ML#1077

Merged
sdrabenh merged 8 commits intofeature/sdrabenh/gcm_v12from
feature/pchakrab/mkiau-pyml-interface
Apr 3, 2025
Merged

Integrate Python code that computes temperature increments via ML#1077
sdrabenh merged 8 commits intofeature/sdrabenh/gcm_v12from
feature/pchakrab/mkiau-pyml-interface

Conversation

@pchakraborty
Copy link
Collaborator

This PR includes an interface in MKIAU to the Python code to compute temperature increments via ML. The relevant code is built via the CMake option BUILD_WITH_PYMLINC (as in cmake -DBUILD_WITH_PYMLINC=On ...). This option sets the compile definition HAS_PYMLINC that activates the interface and the associated code that includes a new export variable DTDT_ML.

The PR passes gcm_regress test and the 0-increment replay test.

…ments

Renamed ...
  BUILD_PYMLINC_INTERFACE -> BUILD_WITH_PYMLINC (cmake option)
  PYMLINC_INTEGRATION -> HAS_PYMLINC (compile definition)
Using BUILD_WITH_PYMLINC at a higher level (GCM), since we need it in AGCM and Physics to re-export the extra Q's and provide connectivity at the GCM level

We need Q[L/I/R/S/G]TOT variables, instead of Q[L/I/R/S/G]

Checking in

Working code

Working code - added U

Working code - added V, T

Working code - added rest of Q's

Working code - added PS. All variables are have now been passed to Python

Calling geos_state_bias code

Checking in working version, before everything starts failing again

Now working

Still working

Still working. Miracle!

Still working

Cleanup step 1

Cleanup step 2

Still working

Working

Working

Working

Only root calls pyMLINC's interface

Working

Working
@pchakraborty pchakraborty requested review from a team as code owners March 16, 2025 16:34
@pchakraborty pchakraborty added the 0 diff The changes in this pull request have verified to be zero-diff with the target branch. label Mar 16, 2025
@sdrabenh sdrabenh merged commit 7bea975 into feature/sdrabenh/gcm_v12 Apr 3, 2025
8 checks passed
@pchakraborty pchakraborty deleted the feature/pchakrab/mkiau-pyml-interface branch May 9, 2025 16:11
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