Skip to content

Commit

Permalink
add standard name of ccpp_t instance for the empty-init case rather t…
Browse files Browse the repository at this point in the history
…han local name to fix bug (#517)

This PR is a bugfix for ufs-community/ccpp-physics#127

#503 was merged to fix the case when no scheme within a group has an init phase. A bug was introduced described in the ccpp-physics issue above. This fix ensures that the ccpp_t_instance variable is only passed once in the caps.
  • Loading branch information
grantfirl authored Jan 11, 2024
1 parent 0eca5c2 commit 87c2b32
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion scripts/mkstatic.py
Original file line number Diff line number Diff line change
Expand Up @@ -1617,7 +1617,7 @@ def write(self, metadata_request, metadata_define, arguments, debug):
#the initialized_set_block for the init phase tries to reference the unavailable ccpp_t variable.
if (ccpp_stage == 'init' and not self.parents[ccpp_stage]):
ccpp_var.intent = 'in'
self.parents[ccpp_stage].update({ccpp_var.local_name:ccpp_var})
self.parents[ccpp_stage].update({ccpp_var.standard_name:ccpp_var})

# Get list of arguments, module use statement and variable definitions for this subroutine (=stage for the group)
(self.arguments[ccpp_stage], sub_module_use, sub_var_defs) = create_arguments_module_use_var_defs(
Expand Down

0 comments on commit 87c2b32

Please sign in to comment.