Conversation
|
@iangrooms Just a quick note, I added |
|
@iangrooms, I added @iangrooms & @gustavo-marques, while at it, should I also switch to the CMOR equivalent names of the following fields listed in diag_table.yaml?:
|
|
@alperaltuntas yes, I think it would be good to switch the fields you mentioned to CMOR now, rather than having to rename them after the fact. |
|
I agree, thank you for pointing out these fields. |
|
@iangrooms @gustavo-marques I made the all the changes, I think. This PR is ready for the next round of reviews. |
iangrooms
left a comment
There was a problem hiding this comment.
Looks good to me. Thanks!
This PR adds the diag table changes listed in #308
Testing: GW_JRA.TL319_t232, B1850C_LTso.ne30_t232