Change Ozone Variable Name and Turn on Error

[danholdaway]

Description

Some small fixes needed for current GMAO testing.

• The variable for instruments like MLS should be ozoneProfile, rather than ozoneTotal.
• Some of our testing relies on use of the GSI initial error. We are working to set the error in the YAML instead but right now the error isn't being set. Either it should be set to something sensible or the variable removed entirely. Here we set it to something sensible.

Issue(s) addressed

N/A

Dependencies

N/A

Impact

N/A

Checklist

• [X] I have performed a self-review of my own code
• [NA] I have made corresponding changes to the documentation
• [X] I have run the unit tests before creating the PR

[karpob]

To match the new convention you'll also need to change ozoneLayer to ozoneTotal in the If statement

[BenjaminRuston]

seeing two failed tests:

iodaconv_gsi_ncdiag_coding_norms
test_iodaconv_gsidiag_rad

--

test_iodaconv_gsidiag_rad

[PatNichols]

@danholdaway do you need to update the output file?

[BenjaminRuston]

update of testOutput likely needed

[danholdaway]

OK thanks. I will take a look.

[karpob]

@danholdaway I think you might need to indent twice for that "if" line continuation coding norm error.

[danholdaway]

I think it's ready now. Thanks all.

[gmao-jjin3]

@PatNichols @BenjaminRuston I think this one needs to be merged ASAP, so that we check it out and re-create our data.

[BenjaminRuston]

@gthompsnJCSDA think this is all set

[BenjaminRuston]

@jeromebarre just FYI, note this was merged please let us know if you see anything that may need attention