Brett Raczka

@mjs2369 I agree the comments are confusing. From what I can tell the DART comments are inherited directly from the WRF code documentation and comments -- which are confusing. The...

> [@mjs2369](https://github.com/mjs2369) I agree the comments are confusing. From what I can tell the DART comments are inherited directly from the WRF code documentation and comments -- which are confusing....

My work directory for testing the GOES obs converter (convert_goes_ABI_L1b) is located here: ``/glade/work/bmraczka/DART/observations/obs_converters/GOES/work/`` An example of a GOES radiance raw data file is located there : ``OR_ABI-L1b-RadF-M6C08_G16_s20211841720210_e20211841729518_c20211841729575.nc``. I can...

> two would be great since the other bug is "bug: GOES converter does not handle multiple input files" Cheers, Helen Done

Sorry -- the input.nml file got reverted to the default. The input.nml I used for the GOES conversion is here: /glade/work/bmraczka/DART/observations/obs_converters/GOES/work/input.nml.GOES_converter

> see default input.nml comment Changes made to make input.nml compatible to run the converter. Should be ready for approval.

Besides documentation, perhaps I could have also forced the code to throw an error if temperature variable ``T`` is used as the ``TYPE_T`` in wrf_state_variables?

> see [#733 (review)](https://github.com/NCAR/DART/pull/733#pullrequestreview-2292522250) Yup, I am on it

Now errors out if using T WRF variable with type_t. Should both prevent boundscheck error, and also an end-user unexpectedly using a diagnostic temperature variable instead of prognostic temperature variable...

> Hi Brett, can you share a wrf restart file to test this code with? Thanks, Helen Yes -- this wrf file has both T and THM to test out...