Skip to content

Hook up SUBDD-State_Diag for 3D StateMet tracers#99

Closed
joewallwork wants to merge 11 commits into
history-rc-file-for-model-efrom
29_subdd-statediag-3d
Closed

Hook up SUBDD-State_Diag for 3D StateMet tracers#99
joewallwork wants to merge 11 commits into
history-rc-file-for-model-efrom
29_subdd-statediag-3d

Conversation

@joewallwork

Copy link
Copy Markdown
Collaborator

Towards #29.
Merges into #84.

This PR hooks up the 3D StateMet fields.

@joewallwork joewallwork added the deliverable1 Associated with deliverable 1 label Mar 14, 2025
Comment thread model/CHEM_DRV.F90 Outdated
Comment on lines +2282 to +2291
! TODO: State_Met%PEDGE

#ifdef CALC_MERRA2_LIKE_DIAGS

! TODO: State_Met%PFICU
! TODO: State_Met%PFILSAN
! TODO: State_Met%PFLCU
! TODO: State_Met%PFLLSAN

#endif

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Oh actually, I don't see these in the HISTORY file so don't think they're needed.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

They appear in the "THE LevelEdgeDiags COLLECTION" rather than "The StateMet COLLECTION" so I guess we leave them out for now?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Ah, now I see

# %%%%% The StateMet COLLECTION %%%%%
#
# Fields of the State_Met object (also see the LevelEdgeDiags collection)

I guess we can discuss on Tuesday.

Comment thread model/CHEM_DRV.F90
Comment thread model/CHEM_DRV.F90 Outdated
Comment on lines +441 to +454
! Dry air mass [kg]
! TODO: State_Met%AD (II,JJ,K) = ???

! Dry air density [kg m-3]
! TODO: State_Met%AIRDEN (II,JJ,K) = ???

! Volume of grid box [m3]
! TODO: State_Met%AIRVOL (II,JJ,K) = ???

! Water vapor mixing ratio (w/r/t dry air)
! TODO: State_Met%AVGW (II,JJ,K) = ???

! Grid box height [m]
! TODO: State_Met%BXHEIGHT (II,JJ,K) = ???

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

These fields are still TODO.

Comment thread model/CHEM_DRV.F90 Outdated
Comment on lines +465 to +469
! Delta-pressure across grid box(wet air) [hPa]
! TODO: State_Met%DELP (II,JJ,K) = ???

! Delta-pressure across grid box (dry air) [hPa]
! TODO: State_Met%DELPDRY (II,JJ,K) = ???

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

These fields are still TODO.

Comment thread model/CHEM_DRV.F90 Outdated
Comment on lines +544 to +545
! Potential temperature [K]
! TODO: State_Met%THETA (II,JJ,K) = ???

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This field is still TODO.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@jwallwork23, let me do these fields right away. Thanks for the heads up!

@joewallwork joewallwork Mar 18, 2025

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Apologies, these TODOs were for myself.

Unfortunately, what you pushed in cb3cba6 won't work because the arrays on the RHS need to come from the GISS model. In each case, we need to:

  1. Find out what the array is called in the GISS model.
  2. Add a USE <module>, ONLY: <array> syntax at the top of the subroutine to import the array.
  3. Add the indexed array on the RHS of the expression.

You'll notice that the coupled model CI builds failed with your commit because the arrays you added haven't been imported (and might not even have those names anyway).

@joewallwork

Copy link
Copy Markdown
Collaborator Author

Closing this PR as superseded by #102.

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

Labels

deliverable1 Associated with deliverable 1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants