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

[BUG/ISSUE] State_Diag%JvalO3O1D and State_Diag%JvalO3O3P were not getting zeroed in flexchem_mod.F90 #810

Closed
yantosca opened this issue Aug 4, 2021 · 1 comment
Assignees
Labels
category: Bug Something isn't working topic: Diagnostics Related to output diagnostic data
Milestone

Comments

@yantosca
Copy link
Contributor

yantosca commented Aug 4, 2021

This issue came out of the discussion in #804.

Lizzie Lundgren wrote:

I quickly looked at the code and it appears the new J-value diagnostics are not set to zero the way the old ones are. See

! Zero diagnostic archival arrays to make sure that we don't have any
! leftover values from the last timestep near the top of the chemgrid
IF (State_Diag%Archive_Loss ) State_Diag%Loss = 0.0_f4
IF (State_Diag%Archive_Prod ) State_Diag%Prod = 0.0_f4
IF (State_Diag%Archive_Jval ) State_Diag%Jval = 0.0_f4
IF (State_Diag%Archive_JNoon ) State_Diag%JNoon = 0.0_f4
IF (State_Diag%Archive_ProdCOfromCH4 ) State_Diag%ProdCOfromCH4 = 0.0_f4
IF (State_Diag%Archive_ProdCOfromNMVOC) State_Diag%ProdCOfromNMVOC= 0.0_f4
IF (State_Diag%Archive_OHreactivity ) State_Diag%OHreactivity = 0.0_f4
IF (State_Diag%Archive_RxnRate ) State_Diag%RxnRate = 0.0_f4
IF (State_Diag%Archive_KppDiags) THEN
IF (State_Diag%Archive_KppIntCounts) State_Diag%KppIntCounts = 0.0_f4
IF (State_Diag%Archive_KppJacCounts) State_Diag%KppJacCounts = 0.0_f4
IF (State_Diag%Archive_KppTotSteps ) State_Diag%KppTotSteps = 0.0_f4
IF (State_Diag%Archive_KppAccSteps ) State_Diag%KppAccSteps = 0.0_f4
IF (State_Diag%Archive_KppRejSteps ) State_Diag%KppRejSteps = 0.0_f4
IF (State_Diag%Archive_KppLuDecomps) State_Diag%KppLuDecomps = 0.0_f4
IF (State_Diag%Archive_KppSubsts ) State_Diag%KppSubsts = 0.0_f4
IF (State_Diag%Archive_KppSmDecomps) State_Diag%KppSmDecomps = 0.0_f4
ENDIF
. That list of is missing the two new arrays.

@yantosca yantosca added the category: Bug Something isn't working label Aug 4, 2021
@yantosca yantosca added this to the 13.2.1 milestone Aug 4, 2021
@yantosca yantosca self-assigned this Aug 4, 2021
@yantosca
Copy link
Contributor Author

yantosca commented Aug 4, 2021

In commit 5cb62c8, we now have added the following code to make sure all relevant diagnostic arrays are zeroed in flexchem_mod.f90.

    IF (State_Diag%Archive_Jval           ) State_Diag%Jval           = 0.0_f4
    IF (State_Diag%Archive_Jval           ) State_Diag%JvalO3O1D      = 0.0_f4
    IF (State_Diag%Archive_Jval           ) State_Diag%JvalO3O3P      = 0.0_f4

This update has been added to the patch/13.2.1 branch. It will ship in version 13.2.1, which will be issued immediately after the 13.2.0 release.

NOTE: This fix does not resolve the issue of why these diagnostics may be lower than in previous versions. For that discussion, please see #804.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category: Bug Something isn't working topic: Diagnostics Related to output diagnostic data
Projects
None yet
Development

No branches or pull requests

1 participant