Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inline post does not send 4D biomass burning (EBB) variable to UPP #846

Closed
SamuelTrahanNOAA opened this issue Jun 12, 2024 · 1 comment · Fixed by #845
Closed

Inline post does not send 4D biomass burning (EBB) variable to UPP #846

SamuelTrahanNOAA opened this issue Jun 12, 2024 · 1 comment · Fixed by #845
Labels
bug Something isn't working

Comments

@SamuelTrahanNOAA
Copy link
Contributor

SamuelTrahanNOAA commented Jun 12, 2024

Description

The newest version of UPP has a 4D variable (in module VRBLS4D) called "EBB" which replaces its prior 2D variable (in module VRBLS2D). FV3's inline post (io/post_fv3.F90) tries to access the 2D array, which no longer exist. This must be corrected before the UPP hash can update.

It appears EBB(:,:,:,1) comes from the ebu_smoke variable. Only the first index (:,:,:,1) of the array is ever used. This change came from @EricJames-NOAA. Nobody made the corresponding change to the inline post. Indeed, they couldn't since there is a library version incompatibility between UPP and ufs-weather-model (see ufs-community/ufs-weather-model#2322).

I think the solution is to send ebu_smoke into ebb(:,:,:,1) in post_fv3.F90.

To Reproduce:

What compilers/machines are you seeing this with?
Give explicit steps to reproduce the behavior.

  1. Update the upp submodule to the head of UPP's develop branch.
  2. Update the ufs-weather-model to use the upp-addon-env Spack Stack.
  3. Try to compile with the Intel compiler on Hera. (Spack Stack has been updated there.)
  4. Witness the errors in accessing the EBB variable in post_fv3.F90

Additional context

This problem came up when trying to add new fields to the inline post. To do that, we must update the UPP submodule hash to the head of develop. After I got past the library incompatibility, I ran into this EBB trouble.

Output

N/A

Testing:

No code changes yet.

Dependent PRs:

N/A

@SamuelTrahanNOAA SamuelTrahanNOAA added the bug Something isn't working label Jun 12, 2024
@SamuelTrahanNOAA
Copy link
Contributor Author

Pinging @WenMeng-NOAA and @DusanJovic-NOAA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant