Difference between revisions of "Minutes of the code management group meeting 2019-09-16"

From UKCA
 
(27 intermediate revisions by 4 users not shown)
Line 8: Line 8:
 
* '''Participant PIN: ''' 735112
 
* '''Participant PIN: ''' 735112
   
  +
'''Met Office:''' JH, MD, AJH, AH, MS, LN
==Present==
 
   
'''Met Office:'''
+
'''Cambridge:''' LA, SAN
   
'''Cambridge:'''
+
'''Leeds:''' DG, WF, MR, KP
 
'''Leeds:'''
 
   
 
'''Reading:'''
 
'''Reading:'''
Line 24: Line 22:
 
'''UEA:'''
 
'''UEA:'''
   
'''NIWA:'''
+
'''NIWA:''' OM
   
 
'''CSIRO:'''
 
'''CSIRO:'''
   
'''Apologies:'''
+
'''Apologies:''' HG, FOC, AL, NB
   
 
==Actions from this meeting ==
 
==Actions from this meeting ==
   
  +
# (ongoing) LA to put <code>C_SPECIES</code> definitions in the ''Redundant Code'' table (short-term), and switch to use <code>M_SPECIES</code> instead. Talk with JH about AQUM tests and requirements. Note that there will be problems with exoplanets and >100km top configs where M_air not constant. LA to make a branch and progress for vn11.6.
  +
# (ongoing) LA check MD's script for perpetual nudging.
  +
# (ongoing) FOC to see what action can be taken from MASS, then put out an email for POCs from institutes. FOC/AH/MD to develop a strategy on MASS for discussion at next CMG as a data clear-out is required.
  +
# (ongoing) JM to find someone to progress ticket/bugfix for SO2 deposition. Will need to be on a switch for UKESM1. LA to follow-up with JM
  +
# (ongoing) LA to contact AS/AJH and ask how best to proceed with removing the depreciated NetCDF attribute options. Do after GA8 is released as still required for GA7.
  +
# (ongoing) FOC to encourage Garry Hayman to start lodging JULES code piecemeal.
  +
# (ongoing) '''ALL''' to read API document and look through new API code and send comments to JH
  +
# (ongoing) GM (supported by KP/MY/AH) to make table on UKCA wiki documenting this. LA to email.
  +
# (ongoing) Those with access check ukca_bugs regularly via OWA until Outlook access is fixed.
  +
# (ongoing) LA to contact MH/OW regarding FastJX-GLOMAP coupling. Follow-up after UMvn11.5 is released.
  +
# (ongoing) MD to update um_ukca_eg app to be more in line with UKESM1, and add a rigorous test. Age-of-air to be added.
  +
# (ongoing) LA to add N48 Linux and update EXP test
  +
# (ongoing) '''ALL''' to consider other rose-stem tests that could be included
  +
# (ongoing) '''ALL''' to email LA with suggestions for next set of tutorials
  +
# (new) LA to ask GF to add more details to ticket [https://code.metoffice.gov.uk/trac/um/ticket/4913 UM:#4913] to understand the problem (bug #109)
  +
# (new) '''ALL''' to consider thinking about fixing bug #113 (see [https://code.metoffice.gov.uk/trac/um/ticket/5119 UM:#5119])
  +
# (new) GM to email AH/KC regarding nitrate
  +
# (new) '''ALL''' to make changes to [[Testing Requirements for UKCA Code Changes]] wiki page as they see fit.
  +
# (new) MR & LA to talk offline regarding Chris Dearden's tickets.
  +
# (new) LA to contact AJH/GM offline regarding coupling StratTrop to GLOMAP-CLIM
  +
# (new) LA to send message to Paul Agnew regarding an AQUM representative to these meetings
  +
# (new) LA to send Doodle poll for early November.
  +
#* https://doodle.com/poll/nteifwn2frz2ngx4
  +
  +
=== Standing actions ===
  +
  +
# (ongoing - standing item) '''CMG''' to monitor list of bugs and circulate between group for owners.
  +
# (ongoing - standing item) At '''CMG''' future meetings there should be included an appendix listing the open bugs. If a ticket is included it is not an open bug (as the owner of ticket is person doing lodging).
  +
# (ongoing - standing item) '''ALL''' with access to keep monitoring ukca_bugs email list.
  +
# (ongoing - standing item) '''ALL''' to think about if code changes will change KGO and let systems team know in good time, and remember to include the UKCA & SC0138 keywords to all UKCA tickets. This should be a standard part of our working practice. ''Note'' also the required keywords.
  +
# (ongoing - standing item) '''ALL''' to note, review, and remove obsolete/redundant code'''
  +
# (ongoing - standing item) '''ALL''' closed ticket owners to produce a summary of their change for the UKCA newsletter for special occasions, e.g. to coincide with Composition-Climate Meeting.
  +
# (ongoing - standing item) '''Code Owner''' to produce draft newsletter for '''CMG''' to approve.
   
 
== Minutes ==
 
== Minutes ==
Line 40: Line 71:
   
 
# (ongoing) LA to put <code>C_SPECIES</code> definitions in the ''Redundant Code'' table (short-term), and switch to use <code>M_SPECIES</code> instead. Talk with JH about AQUM tests and requirements. Note that there will be problems with exoplanets and >100km top configs where M_air not constant. LA to make a branch and progress for vn11.6.
 
# (ongoing) LA to put <code>C_SPECIES</code> definitions in the ''Redundant Code'' table (short-term), and switch to use <code>M_SPECIES</code> instead. Talk with JH about AQUM tests and requirements. Note that there will be problems with exoplanets and >100km top configs where M_air not constant. LA to make a branch and progress for vn11.6.
  +
#* '''Ongoing'''
 
# (ongoing) LA check MD's script for perpetual nudging.
 
# (ongoing) LA check MD's script for perpetual nudging.
  +
#* '''Ongoing'''
 
# (ongoing) LA to create Wiki page on UKCA website for users to add details of configurations and input files.
 
# (ongoing) LA to create Wiki page on UKCA website for users to add details of configurations and input files.
  +
#* '''Closed'''
 
# (new) LA to add ''All Pages'' link on UKCA Wiki
 
# (new) LA to add ''All Pages'' link on UKCA Wiki
  +
#* '''Closed'''
 
# (ongoing) LA to discuss with Leeds plans for a separate repository.
 
# (ongoing) LA to discuss with Leeds plans for a separate repository.
  +
#* Videoconference held on 2019-06-24. General feelings were that it was a positive move.
  +
#* '''Closed'''
 
# (ongoing) FOC to see what action can be taken from MASS, then put out an email for POCs from institutes. FOC/AH/MD to develop a strategy on MASS for discussion at next CMG as a data clear-out is required.
 
# (ongoing) FOC to see what action can be taken from MASS, then put out an email for POCs from institutes. FOC/AH/MD to develop a strategy on MASS for discussion at next CMG as a data clear-out is required.
  +
#* No strategy developed yet.
# (ongoing) JM to find someone to progress ticket/bugfix for SO2 deposition. Will need to be on a switch for UKESM1
 
  +
#* '''Ongoing'''
  +
# (ongoing) JM to find someone to progress ticket/bugfix for SO2 deposition. Will need to be on a switch for UKESM1
  +
#* LA to follow-up with JM
  +
#* '''Ongoing'''
 
# (ongoing) LA to contact AS/AJH and ask how best to proceed with removing the depreciated NetCDF attribute options.
 
# (ongoing) LA to contact AS/AJH and ask how best to proceed with removing the depreciated NetCDF attribute options.
  +
#* From AS: ''In the ancil ticket linked from #2513 (https://code.metoffice.gov.uk/trac/ancil/ticket/782) it says that GA7 depends on the old format files, and as far as I know this is still true. In the 3-step timeline suggested, we haven't reached step 2 unfortunately. That ticket was about ensuring GA8 did not use the old attributes. I believe that GA8 is due for release in the next few months (it has been a very long time coming), so UM vn11.5 may be the last version at which GA7 is supported. Do you mind waiting a bit longer?''
  +
#* '''Ongoing'''
 
# (ongoing) FOC to encourage Garry Hayman to start lodging JULES code piecemeal.
 
# (ongoing) FOC to encourage Garry Hayman to start lodging JULES code piecemeal.
  +
#* '''Ongoing'''
 
# (ongoing) '''ALL''' to read API document and look through new API code and send comments to JH
 
# (ongoing) '''ALL''' to read API document and look through new API code and send comments to JH
  +
#* Timescales have slipped somewhat. Document will be updated after next release.
  +
#* '''Ongoing'''
 
# (ongoing) LA to email GM and ask how GLOMAP would differ from how it is in other models (e.g. CTM, IFS etc.)
 
# (ongoing) LA to email GM and ask how GLOMAP would differ from how it is in other models (e.g. CTM, IFS etc.)
  +
#* '''Ongoing'''
  +
#* What was not clear to the group was how the GLOMAP versioning system related to UM versions & versions of GLOMAP in other models, and whether there were developments that could easily be incorporated from elsewhere.
  +
#* '''ACTION:''' GM (supported by KP/MY/AH) to make table on UKCA wiki documenting this. LA to email.
 
# (ongoing) LA to send priorities list to UKCA SMG
 
# (ongoing) LA to send priorities list to UKCA SMG
 
## UKCA API
 
## UKCA API
Line 68: Line 117:
 
## GLOMAP-CLIM API refresh
 
## GLOMAP-CLIM API refresh
 
## N48L85 configuration
 
## N48L85 configuration
  +
#* '''Closed'''
 
# (new) Those with access check ukca_bugs regularly via OWA until Outlook access is fixed.
 
# (new) Those with access check ukca_bugs regularly via OWA until Outlook access is fixed.
  +
#* '''Ongoing'''
 
# (new) LA to take ownership of bug #107
 
# (new) LA to take ownership of bug #107
  +
#* '''Closed'''
 
# (new) LA to contact MH/OW regarding FastJX-GLOMAP coupling
 
# (new) LA to contact MH/OW regarding FastJX-GLOMAP coupling
  +
#* Ongoing email discussions. Last from MH on 2019-06-19: ''"Thanks for the update on getting the FAST-JX GLOMAP coupling into the code. I will try my best to get it into the trunk at version 11.6. I might take you up on your kind offer to help with the tests when needed if that is ok?"''
# (new) MD to update um_ukca_eg app to be more in line with UKESM1, and add a rigorous test
 
  +
#* '''Ongoing''' - follow-up after UMvn11.5 is released.
  +
# (new) MD to update um_ukca_eg app to be more in line with UKESM1, and add a rigorous test. ''An Age-of-air-only configuration being added due to other changes breaking this at vn11.4. Will still attempt to update the app if possible, otherwise bump to vn11.6''.
  +
#* '''Ongoing'''
 
# (new) LA to add N48 Linux and update EXP test
 
# (new) LA to add N48 Linux and update EXP test
  +
#* '''Ongoing'''
 
# (new) '''ALL''' to consider other rose-stem tests that could be included
 
# (new) '''ALL''' to consider other rose-stem tests that could be included
  +
#* There is a new GLOMAP-CLIM test. Plans for a UKCA-CASIM test.
# (new) LA to check-in with Rich Gilham after the Code Freeze
 
  +
# (new) LA to check-in with Rich Gilham after the Code Freeze regarding SC0138 keyword
  +
#* Discussed at [https://code.metoffice.gov.uk/trac/um/wiki/ProjectDocumentation/2019-07/Minutes UMPB July 2019]: ''General discussion if the SC0138 keyword is really necessary and if replacing it with an appropriate collab keyword. Final verdict: Time to stop using it (SC0138 keyword).''
  +
#* '''Closed'''
 
# (new) '''ALL''' to email LA with suggestions for next set of tutorials
 
# (new) '''ALL''' to email LA with suggestions for next set of tutorials
  +
#* '''Ongoing'''
 
# (new) LA to send Doodle poll for early September.
 
# (new) LA to send Doodle poll for early September.
 
#* https://doodle.com/poll/fn88usa3pb97xbuz
 
#* https://doodle.com/poll/fn88usa3pb97xbuz
  +
#* '''Closed'''
   
 
==== Standing actions ====
 
==== Standing actions ====
Line 100: Line 161:
 
!style="text-align:top;rowspan=2";|Comments from this meeting
 
!style="text-align:top;rowspan=2";|Comments from this meeting
 
|-
 
|-
| 35 || Error in labelling: Surface resistance <--> stomatal resistance ? || [https://code.metoffice.gov.uk/trac/um/ticket/1409 UM:#1409] || '''Fiona O'Connor''' || Leave open. Garry Hayman started work in offline JULES. <br/> Still ongoing. JM found bug in SO2 deposition, where there is an inconsistency when changing tiles. Making progress. <br/> Leave open. GH gave update, has done a lot of work. Concerned it might not be bit-reproducable, but could be solvable. Encouraging to lodge existing. Resistances done as metadata. Not clear as to timescales.<br/> FOC to encourage Garry Hayman to start lodging JULES code piecemeal. ||
+
| 35 || Error in labelling: Surface resistance <--> stomatal resistance ? || [https://code.metoffice.gov.uk/trac/um/ticket/1409 UM:#1409] || '''Fiona O'Connor''' || Leave open. Garry Hayman started work in offline JULES. <br/> Still ongoing. JM found bug in SO2 deposition, where there is an inconsistency when changing tiles. Making progress. <br/> Leave open. GH gave update, has done a lot of work. Concerned it might not be bit-reproducable, but could be solvable. Encouraging to lodge existing. Resistances done as metadata. Not clear as to timescales.<br/> FOC to encourage Garry Hayman to start lodging JULES code piecemeal. || Leave open
 
|-
 
|-
| 101 || Errors in reaction rates and diagnostics || [https://code.metoffice.gov.uk/trac/um/ticket/4466 UM:#4466] || '''Luke Abraham / Scott Archer-Nicolls''' || There is a need to conserve science config for UKESM1, so changes need to be on a flag.<br/>Colin has been looking at DMSO, as this is not currently used in StratTrop but is in Offline.<br/>Might be able to merge the rate with CEJ changes, but best to leave the diagnostic issues as a separate change. <br/> LA, JM, CEJ, SAN to discuss offline and keep GM in loop.<br/>Use versioning system. Talk about further offline. <br/> Keep ongoing. <br/> LA to do with #103 ||
+
| 101 || Errors in reaction rates and diagnostics || [https://code.metoffice.gov.uk/trac/um/ticket/4466 UM:#4466] || '''Luke Abraham / Scott Archer-Nicolls''' || There is a need to conserve science config for UKESM1, so changes need to be on a flag.<br/>Colin has been looking at DMSO, as this is not currently used in StratTrop but is in Offline.<br/>Might be able to merge the rate with CEJ changes, but best to leave the diagnostic issues as a separate change. <br/> LA, JM, CEJ, SAN to discuss offline and keep GM in loop.<br/>Use versioning system. Talk about further offline. <br/> Keep ongoing. <br/> LA to do with #103 || Do for vn11.6. Quite a large ticket.
 
|-
 
|-
| 103 || Incorrect indexing of tmprc array: ASAD_SPIMPMJP || [https://code.metoffice.gov.uk/trac/um/ticket/4562 UM:#4562] || '''Scott Archer-Nicolls''' || Breaks KGO - affects solver convergence. In practice doesn’t make a huge difference, but should go into vn11.4, but does change KGO. UKESM1 - small change might be OK - get in touch for advice - might be able to test with UKESM1 AMIP only. Check with Oliver Wild <br/> Could be rolled-up with #101 <br/> LA to do with #101 ||
+
| 103 || Incorrect indexing of tmprc array: ASAD_SPIMPMJP || [https://code.metoffice.gov.uk/trac/um/ticket/4562 UM:#4562] || '''Scott Archer-Nicolls''' || Breaks KGO - affects solver convergence. In practice doesn’t make a huge difference, but should go into vn11.4, but does change KGO. UKESM1 - small change might be OK - get in touch for advice - might be able to test with UKESM1 AMIP only. Check with Oliver Wild <br/> Could be rolled-up with #101 <br/> LA to do with #101 || Do with #101
 
|-
 
|-
| 104 || Incorrect value for Rc for SO_2 deposition to open water || [https://code.metoffice.gov.uk/trac/um/ticket/4635 UM:#4635] || '''Catherine Hardacre''' || Got to be separate to #35, unless there is an idea of timelines.<br/> JM to find someone to progress bug #104. ||
+
| 104 || Incorrect value for Rc for SO_2 deposition to open water || [https://code.metoffice.gov.uk/trac/um/ticket/4635 UM:#4635] || '''Catherine Hardacre''' || Got to be separate to #35, unless there is an idea of timelines.<br/> JM to find someone to progress bug #104. || LA to follow-up with JM
 
|-
 
|-
| 107 || Missing c_h2o conversion factor in offline_oxidants_be || [https://code.metoffice.gov.uk/trac/um/ticket/4714 um:#4714] || || Code is <code>zq(:)=RESHAPE(q(:,:,k),(/theta_field_size/))</code> rather than <code>zq(:) = RESHAPE(q(:,:,k),(/theta_field_size/))/c_h2o</code> <br/> LA to take ownership and assess ||
+
| 107 || Missing c_h2o conversion factor in offline_oxidants_be || <del>[https://code.metoffice.gov.uk/trac/um/ticket/4714 um:#4714]</del><br/>[https://code.metoffice.gov.uk/trac/um/ticket/5118 um:#5118] || '''Luke Abraham''' || Code is <code>zq(:)=RESHAPE(q(:,:,k),(/theta_field_size/))</code> rather than <code>zq(:) = RESHAPE(q(:,:,k),(/theta_field_size/))/c_h2o</code> <br/> LA to take ownership and assess ||
 
|-
 
|-
 
| 108 || Incorrect stash processing of GLOMAP ''load*, conc* and plume_scav'' diagnostics || [https://code.metoffice.gov.uk/trac/um/ticket/4909 UM:#4909] || '''Mohit Dalvi''' || To be Lodged after 10.6 <br/> Do after CMIP6 runs are finished (after Jan 2020) ||
 
| 108 || Incorrect stash processing of GLOMAP ''load*, conc* and plume_scav'' diagnostics || [https://code.metoffice.gov.uk/trac/um/ticket/4909 UM:#4909] || '''Mohit Dalvi''' || To be Lodged after 10.6 <br/> Do after CMIP6 runs are finished (after Jan 2020) ||
 
|-
 
|-
| 109 || Bug in dry deposition of CH4 for 13/17/27 tiles || [https://code.metoffice.gov.uk/trac/um/ticket/4913 UM:#4913] || '''Gerd Folberth''' || To be Lodged after 10.6 <br/> Gerd noted unusual CH4 output in CMIP6 runs. <br/> May need a rose stem test for H2 and CH4 drdydep code lodged at [https://code.metoffice.gov.uk/trac/um/ticket/4157 UM:#4157] <br/> Makes a huge difference to CH4. Needs testing. AJH or GF will do during vn11.4, progress for vn11.6. May need UKESM1 approval. ||
+
| 109 || Bug in dry deposition of CH4 for 13/17/27 tiles || [https://code.metoffice.gov.uk/trac/um/ticket/4913 UM:#4913] || '''Gerd Folberth''' || To be Lodged after 10.6 <br/> Gerd noted unusual CH4 output in CMIP6 runs. <br/> May need a rose stem test for H2 and CH4 drdydep code lodged at [https://code.metoffice.gov.uk/trac/um/ticket/4157 UM:#4157] <br/> Makes a huge difference to CH4. Needs testing. AJH or GF will do during vn11.4, progress for vn11.6. May need UKESM1 approval. || Code is difficult to understand. AJH thinks GarryH is aware of this problem.<br/>
  +
'''ACTION:''' LA to ask GF to add more details to ticket [https://code.metoffice.gov.uk/trac/um/ticket/4913 UM:#4913] to understand the problem (bug #109)
  +
|-
  +
| 110 || UKCA solar-cycle routine does not pass NRUN-CRUN test || [https://code.metoffice.gov.uk/trac/um/ticket/4828 UM:#4828] || '''Luke Abraham''' || To be Lodged after 11.5 || Have a solution.
  +
|-
  +
| 111 || Cloud fraction array (''cloud_frac'') passed with wrong number of levels </br> Array passed as (row_length, rows, :), so offset by 1 level under ENDGame || Part of [https://code.metoffice.gov.uk/trac/um/ticket/4822 UM:#4822] || '''John Hemmings''' || Affects all ENDGame versions. To be Lodged for 11.5 || Do as separate ticket after #4822 committed. MD #5009 testing
  +
|-
  +
| 112 || <tt>delh2so4_chem</tt> is calculated incorrectly || [https://code.metoffice.gov.uk/trac/um/ticket/5076 UM:#5076] || '''Luke Abraham''' || To be Lodged for 11.6. <br/> Incorrectly "fixed" in [https://code.metoffice.gov.uk/trac/um/ticket/3245 UM:#3245 (Bug #87)]. <br/> All StratTrop+GLOMAP versions post vn8.2. Made worse post-vn11.4 if logical <tt> l_fix_ukca_h2so4_ystore</tt> is True. ||
  +
|-
  +
| 113 || Latitude bugs affecting LAM configurations || [https://code.metoffice.gov.uk/trac/um/ticket/5119 UM:#5119] || '''John Hemmings''' || To be lodged for 11.6 || '''ACTION:''' '''ALL''' to consider thinking about fixing bug #113 (see [https://code.metoffice.gov.uk/trac/um/ticket/5119 UM:#5119])
 
|-
 
|-
 
|}
 
|}
Line 139: Line 209:
 
| 5 || Backward Euler Standard Tropospheric Chemistry || [https://code.metoffice.gov.uk/trac/um/ticket/1507 UM:#1507] || None || '''PROPOSED''' || Medium-term || Fiona O'Connor || ||
 
| 5 || Backward Euler Standard Tropospheric Chemistry || [https://code.metoffice.gov.uk/trac/um/ticket/1507 UM:#1507] || None || '''PROPOSED''' || Medium-term || Fiona O'Connor || ||
 
|-
 
|-
| 6 || Wet oxidation of SO2 in GLOMAP || [https://code.metoffice.gov.uk/trac/um/ticket/4475 UM:#4475] || None || '''PROPOSED''' || Medium-term || Colin Johnson || JM to look at this re: UKESM1 c.f. offline GLOMAP. JM to take ownership. ||
+
| 6 || Wet oxidation of SO2 in GLOMAP || [https://code.metoffice.gov.uk/trac/um/ticket/4475 UM:#4475] || None || '''PROPOSED''' || Medium-term || Colin Johnson || JM to look at this re: UKESM1 c.f. offline GLOMAP. JM to take ownership. || Still needed for CTM.
  +
Relevance for standalone code and later coupling to CTM.
 
|-
 
|-
 
| 7 || Updraft velocity - l_conv_tke || [https://code.metoffice.gov.uk/trac/um/ticket/3668 UM:#3668] || Proposed in December 2017. Approval being sought for GA series (likely GA8). || '''PROPOSED''' || Medium-term || Alan J Hewitt / Ian Boutle|| ||
 
| 7 || Updraft velocity - l_conv_tke || [https://code.metoffice.gov.uk/trac/um/ticket/3668 UM:#3668] || Proposed in December 2017. Approval being sought for GA series (likely GA8). || '''PROPOSED''' || Medium-term || Alan J Hewitt / Ian Boutle|| ||
Line 145: Line 216:
 
| 8 || All backward Euler chemistry schemes || None || Note that this is only for the “hard-coded” BE schemes. Quite a lot of work would be involved, but good as a longer-term aim. ''Note'' that problems for AQUM would need to be addressed first. || '''PROPOSED''' || Long-term || Luke Abraham || ||
 
| 8 || All backward Euler chemistry schemes || None || Note that this is only for the “hard-coded” BE schemes. Quite a lot of work would be involved, but good as a longer-term aim. ''Note'' that problems for AQUM would need to be addressed first. || '''PROPOSED''' || Long-term || Luke Abraham || ||
 
|-
 
|-
| 9 || Deprecated UKCA netcdf attribute options || [https://code.metoffice.gov.uk/trac/um/ticket/2513 UM:#2513] || 'emission_type', string type attributes no longer recommeneded. Need some time for most new code to adopt these. Still used in AQUM files. || '''DEPRECATED''' || Medium-term || Nick Savage || AQUM are addressing the files. Have gone from operational jobs, have ticket to remove from rose-stem. LN to document. These have been removed from AQUM.<br/>AQUM work has been done. ||
+
| 9 || Deprecated UKCA netcdf attribute options || [https://code.metoffice.gov.uk/trac/um/ticket/2513 UM:#2513] || 'emission_type', string type attributes no longer recommeneded. Need some time for most new code to adopt these. Still used in AQUM files. || '''DEPRECATED''' || Medium-term || Nick Savage || AQUM are addressing the files. Have gone from operational jobs, have ticket to remove from rose-stem. LN to document. These have been removed from AQUM.<br/>AQUM work has been done. || From AS: ''In the ancil ticket linked from #2513 (https://code.metoffice.gov.uk/trac/ancil/ticket/782) it says that GA7 depends on the old format files, and as far as I know this is still true. In the 3-step timeline suggested, we haven't reached step 2 unfortunately. That ticket was about ensuring GA8 did not use the old attributes. I believe that GA8 is due for release in the next few months (it has been a very long time coming), so UM vn11.5 may be the last version at which GA7 is supported. Do you mind waiting a bit longer?''
 
|-
 
|-
 
| 10 || ASAD families chemistry || None || Families chemistry has not been used for many years (since at least UM6.1. It is likely that it no longer works if turned on, and it complicates the code making other developments harder || '''DEPRECATED''' || Short-term || Luke Abraham || ||
 
| 10 || ASAD families chemistry || None || Families chemistry has not been used for many years (since at least UM6.1. It is likely that it no longer works if turned on, and it complicates the code making other developments harder || '''DEPRECATED''' || Short-term || Luke Abraham || ||
Line 163: Line 234:
 
| 18 || Remove temporary logical l_fix_neg_pvol_wat <br/> Introduced at [https://code.metoffice.gov.uk/trac/um/ticket/4383 UM:#4383] || [https://code.metoffice.gov.uk/trac/um/ticket/4416 UM:#4416] || Approval being sought for GA series (likely GA9) <br/> [https://code.metoffice.gov.uk/trac/gmed/ticket/421 GMED:#421] || '''PROPOSED''' || Medium-term || Alan J Hewitt || ||
 
| 18 || Remove temporary logical l_fix_neg_pvol_wat <br/> Introduced at [https://code.metoffice.gov.uk/trac/um/ticket/4383 UM:#4383] || [https://code.metoffice.gov.uk/trac/um/ticket/4416 UM:#4416] || Approval being sought for GA series (likely GA9) <br/> [https://code.metoffice.gov.uk/trac/gmed/ticket/421 GMED:#421] || '''PROPOSED''' || Medium-term || Alan J Hewitt || ||
 
|-
 
|-
| 20 || Remove MODULEs/SUBROUTINEs/FUNCTIONs <br/> within UKCA not called within the UM: <br/> <br/> MODULE asad_emissn_mod and SUBROUTINE asad_emissn <br/> MODULE asad_inemit_mod and SUBROUTINE asad_inemit || [https://code.metoffice.gov.uk/trac/um/ticket/4827 UM:#4827] || Requires removing additional redundant emission code from ukca_chem_master.F90 [https://code.metoffice.gov.uk/trac/um/log/main/branches/dev/alanjhewitt/vn11.3_remove_asad_emissn_mod vn11.3_remove_asad_emissn_mod] || '''DEPRECATED''' || Short-term || Alan J Hewitt || Should be finished for vn11.4 ||
+
| 20 || Remove MODULEs/SUBROUTINEs/FUNCTIONs <br/> within UKCA not called within the UM: <br/> <br/> MODULE asad_emissn_mod and SUBROUTINE asad_emissn <br/> MODULE asad_inemit_mod and SUBROUTINE asad_inemit || [https://code.metoffice.gov.uk/trac/um/ticket/4827 UM:#4827] || Requires removing additional redundant emission code from ukca_chem_master.F90 [https://code.metoffice.gov.uk/trac/um/log/main/branches/dev/alanjhewitt/vn11.3_remove_asad_emissn_mod vn11.3_remove_asad_emissn_mod] || '''DEPRECATED''' || Short-term || Alan J Hewitt || Should be finished for vn11.4 || Done. '''Closed'''
 
|-
 
|-
 
| 21 || Remove code behind L_use_stratclim from ukca_fastjx_mod <br/> Introduced prior to MOSRS || [https://code.metoffice.gov.uk/trac/um/ticket/4875 UM:#4875] || This could either be properly included via a namelist or removed from the UM || '''PROPOSED''' || Medium-term || Alan J Hewitt || LA contact OW about progressing FastJX-RADAER work ||
 
| 21 || Remove code behind L_use_stratclim from ukca_fastjx_mod <br/> Introduced prior to MOSRS || [https://code.metoffice.gov.uk/trac/um/ticket/4875 UM:#4875] || This could either be properly included via a namelist or removed from the UM || '''PROPOSED''' || Medium-term || Alan J Hewitt || LA contact OW about progressing FastJX-RADAER work ||
Line 170: Line 241:
 
|-
 
|-
 
|}
 
|}
 
   
 
=== Update on UKCA API redesign ===
 
=== Update on UKCA API redesign ===
Line 176: Line 246:
 
JH has been working on redesigning the UM-UKCA interface to remove the model specific coupling. His proposed design document can be found here:
 
JH has been working on redesigning the UM-UKCA interface to remove the model specific coupling. His proposed design document can be found here:
 
* {{pdf|UKCA_API_design_proposal_v4.pdf|UKCA API design proposal}}
 
* {{pdf|UKCA_API_design_proposal_v4.pdf|UKCA API design proposal}}
  +
  +
JH: Focussing on structural tickets. Good progress made. Working on tracer handling and the environmental drivers ticket & trace gas mix ratios. STASH Tag98 still needed but now purely on the UM side.
  +
MD: Have been working on emissions. Emissions working without changing answers.
  +
  +
=== Progress with Priorities (LA) ===
  +
  +
The current priorities are:
  +
# UKCA API
  +
# Review UKCA tests to cover all requirements
  +
# Fix for <tt>c_h2o</tt> in Offline-Oxidants (bug #107)
  +
# Updates/fixes to GLOMAP routines
  +
# Standalone UKCA configurations
  +
# Emissions flexibility
  +
# Separare UKCA repository
  +
# ACTIVATE optimisation
  +
# FastJX-GLOMAP coupling
  +
# Cloud-J
  +
# Fixes for chemistry rates and ASAD (bugs #101 & #103)
  +
# Rc fixes for water (bug #104)
  +
# CH4 deposition fixes (bug #109)
  +
# UKCA STASHmaster PP-codes
  +
# Removal of redundant code
  +
# GLOMAP-CLIM API refresh
  +
# N48L85 configuration
  +
  +
GM: Highlighted the HyDiS gas-particle partitioning module for nitrate aerosol (with ammonium and interactive pH), currently on a vn7.3 branch.
  +
AH: strategically nitrate is there, raise with AH and/or KC.
  +
  +
'''ACTION:''' GM to email AH/KC regarding nitrate
  +
  +
=== UKCA Testing Requirements (LA) ===
  +
  +
It would be helpful to fully document what is expected regarding code changes, e.g. <tt>rose-stem</tt>, AMIP & short tests, AQUM tests etc.
  +
  +
Initial details have been added here: [[Testing Requirements for UKCA Code Changes]]
  +
  +
'''ACTION:''' '''ALL''' to make changes to [[Testing Requirements for UKCA Code Changes]] wiki page as they see fit.
   
 
=== UM11.5 & UM11.6 Tickets ===
 
=== UM11.5 & UM11.6 Tickets ===
   
See [hhttps://code.metoffice.gov.uk/trac/um/query?status=approved&status=assigned&status=closed&status=code_review&status=committed&status=in_progress&status=new&status=sci%2Ftech_review&keywords=~UKCA&milestone=UM11.5+code+release&col=id&col=summary&col=milestone&col=status&col=type&col=component&order=priority here] for UM vn11.5 tickets on MOSRS.
+
See [https://code.metoffice.gov.uk/trac/um/query?status=approved&status=assigned&status=closed&status=code_review&status=committed&status=in_progress&status=new&status=sci%2Ftech_review&keywords=~UKCA&milestone=UM11.5+code+release&group=owner&col=id&col=summary&col=milestone&col=status&col=type&col=component&order=priority here] for UM vn11.5 tickets on MOSRS.
   
See [https://code.metoffice.gov.uk/trac/um/query?status=approved&status=assigned&status=closed&status=code_review&status=committed&status=in_progress&status=new&status=sci%2Ftech_review&keywords=~UKCA&milestone=UM11.6+code+release&col=id&col=summary&col=milestone&col=status&col=type&col=component&order=priority here] for UM vn11.6 tickets on MOSRS.
+
See [https://code.metoffice.gov.uk/trac/um/query?status=approved&status=assigned&status=closed&status=code_review&status=committed&status=in_progress&status=new&status=sci%2Ftech_review&keywords=~UKCA&milestone=UM11.6+code+release&group=owner&col=id&col=summary&col=milestone&col=status&col=type&col=component&order=priority here] for UM vn11.6 tickets on MOSRS.
   
 
'''PLEASE REMEMBER TO USE THE <tt>UKCA</tt> KEYWORD FOR ALL UKCA CHANGES'''
 
'''PLEASE REMEMBER TO USE THE <tt>UKCA</tt> KEYWORD FOR ALL UKCA CHANGES'''
   
 
'''UM vn11.5 is a CLOSED RELEASE for UKCA changes''' (except for API changes).
 
'''UM vn11.5 is a CLOSED RELEASE for UKCA changes''' (except for API changes).
  +
  +
MR: Discussed inheriting Chris Dearden’s tickets. LA to talk offline
  +
  +
'''ACTION:''' MR & LA to talk offline regarding Chris Dearden's tickets.
   
 
=== AOB/DNM (LA) ===
 
=== AOB/DNM (LA) ===
  +
  +
AJH - planned refactoring work to get benefits of OpenMP (removing IF inside DO loops). See [https://code.metoffice.gov.uk/trac/um/log/main/branches/dev/alanjhewitt/r71623_refactor_gc_ukca changes to ukca_mode_setup.F90].
  +
  +
'''ACTION:''' LA to contact AJH/GM offline regarding coupling StratTrop to GLOMAP-CLIM
  +
  +
LN: Will be going on maternity leave, and so will need a replacement AQUM representative for these meetings.
  +
  +
'''ACTION:''' LA to send message to Paul Agnew regarding an AQUM representative to these meetings
   
 
=== Date of next meeting (LA) ===
 
=== Date of next meeting (LA) ===
  +
  +
'''ACTION:''' LA to send Doodle poll for early November.
  +
  +
* https://doodle.com/poll/nteifwn2frz2ngx4

Latest revision as of 13:14, 18 September 2019

Code management group minutes

Teleconference Details

Monday 16th September 2019, 10am - 11.30am GMT

  • UK Landline: +44 (0)1223 761376 (x61376 from an internal University of Cambridge phone)
  • Participant PIN: 735112

Met Office: JH, MD, AJH, AH, MS, LN

Cambridge: LA, SAN

Leeds: DG, WF, MR, KP

Reading:

Lancaster:

Edinburgh:

UEA:

NIWA: OM

CSIRO:

Apologies: HG, FOC, AL, NB

Actions from this meeting

  1. (ongoing) LA to put C_SPECIES definitions in the Redundant Code table (short-term), and switch to use M_SPECIES instead. Talk with JH about AQUM tests and requirements. Note that there will be problems with exoplanets and >100km top configs where M_air not constant. LA to make a branch and progress for vn11.6.
  2. (ongoing) LA check MD's script for perpetual nudging.
  3. (ongoing) FOC to see what action can be taken from MASS, then put out an email for POCs from institutes. FOC/AH/MD to develop a strategy on MASS for discussion at next CMG as a data clear-out is required.
  4. (ongoing) JM to find someone to progress ticket/bugfix for SO2 deposition. Will need to be on a switch for UKESM1. LA to follow-up with JM
  5. (ongoing) LA to contact AS/AJH and ask how best to proceed with removing the depreciated NetCDF attribute options. Do after GA8 is released as still required for GA7.
  6. (ongoing) FOC to encourage Garry Hayman to start lodging JULES code piecemeal.
  7. (ongoing) ALL to read API document and look through new API code and send comments to JH
  8. (ongoing) GM (supported by KP/MY/AH) to make table on UKCA wiki documenting this. LA to email.
  9. (ongoing) Those with access check ukca_bugs regularly via OWA until Outlook access is fixed.
  10. (ongoing) LA to contact MH/OW regarding FastJX-GLOMAP coupling. Follow-up after UMvn11.5 is released.
  11. (ongoing) MD to update um_ukca_eg app to be more in line with UKESM1, and add a rigorous test. Age-of-air to be added.
  12. (ongoing) LA to add N48 Linux and update EXP test
  13. (ongoing) ALL to consider other rose-stem tests that could be included
  14. (ongoing) ALL to email LA with suggestions for next set of tutorials
  15. (new) LA to ask GF to add more details to ticket UM:#4913 to understand the problem (bug #109)
  16. (new) ALL to consider thinking about fixing bug #113 (see UM:#5119)
  17. (new) GM to email AH/KC regarding nitrate
  18. (new) ALL to make changes to Testing Requirements for UKCA Code Changes wiki page as they see fit.
  19. (new) MR & LA to talk offline regarding Chris Dearden's tickets.
  20. (new) LA to contact AJH/GM offline regarding coupling StratTrop to GLOMAP-CLIM
  21. (new) LA to send message to Paul Agnew regarding an AQUM representative to these meetings
  22. (new) LA to send Doodle poll for early November.

Standing actions

  1. (ongoing - standing item) CMG to monitor list of bugs and circulate between group for owners.
  2. (ongoing - standing item) At CMG future meetings there should be included an appendix listing the open bugs. If a ticket is included it is not an open bug (as the owner of ticket is person doing lodging).
  3. (ongoing - standing item) ALL with access to keep monitoring ukca_bugs email list.
  4. (ongoing - standing item) ALL to think about if code changes will change KGO and let systems team know in good time, and remember to include the UKCA & SC0138 keywords to all UKCA tickets. This should be a standard part of our working practice. Note also the required keywords.
  5. (ongoing - standing item) ALL to note, review, and remove obsolete/redundant code
  6. (ongoing - standing item) ALL closed ticket owners to produce a summary of their change for the UKCA newsletter for special occasions, e.g. to coincide with Composition-Climate Meeting.
  7. (ongoing - standing item) Code Owner to produce draft newsletter for CMG to approve.

Minutes

Actions from the Last Meeting (LA)

Minutes of the code management group meeting 2019-06-13

  1. (ongoing) LA to put C_SPECIES definitions in the Redundant Code table (short-term), and switch to use M_SPECIES instead. Talk with JH about AQUM tests and requirements. Note that there will be problems with exoplanets and >100km top configs where M_air not constant. LA to make a branch and progress for vn11.6.
    • Ongoing
  2. (ongoing) LA check MD's script for perpetual nudging.
    • Ongoing
  3. (ongoing) LA to create Wiki page on UKCA website for users to add details of configurations and input files.
    • Closed
  4. (new) LA to add All Pages link on UKCA Wiki
    • Closed
  5. (ongoing) LA to discuss with Leeds plans for a separate repository.
    • Videoconference held on 2019-06-24. General feelings were that it was a positive move.
    • Closed
  6. (ongoing) FOC to see what action can be taken from MASS, then put out an email for POCs from institutes. FOC/AH/MD to develop a strategy on MASS for discussion at next CMG as a data clear-out is required.
    • No strategy developed yet.
    • Ongoing
  7. (ongoing) JM to find someone to progress ticket/bugfix for SO2 deposition. Will need to be on a switch for UKESM1
    • LA to follow-up with JM
    • Ongoing
  8. (ongoing) LA to contact AS/AJH and ask how best to proceed with removing the depreciated NetCDF attribute options.
    • From AS: In the ancil ticket linked from #2513 (https://code.metoffice.gov.uk/trac/ancil/ticket/782) it says that GA7 depends on the old format files, and as far as I know this is still true. In the 3-step timeline suggested, we haven't reached step 2 unfortunately. That ticket was about ensuring GA8 did not use the old attributes. I believe that GA8 is due for release in the next few months (it has been a very long time coming), so UM vn11.5 may be the last version at which GA7 is supported. Do you mind waiting a bit longer?
    • Ongoing
  9. (ongoing) FOC to encourage Garry Hayman to start lodging JULES code piecemeal.
    • Ongoing
  10. (ongoing) ALL to read API document and look through new API code and send comments to JH
    • Timescales have slipped somewhat. Document will be updated after next release.
    • Ongoing
  11. (ongoing) LA to email GM and ask how GLOMAP would differ from how it is in other models (e.g. CTM, IFS etc.)
    • Ongoing
    • What was not clear to the group was how the GLOMAP versioning system related to UM versions & versions of GLOMAP in other models, and whether there were developments that could easily be incorporated from elsewhere.
    • ACTION: GM (supported by KP/MY/AH) to make table on UKCA wiki documenting this. LA to email.
  12. (ongoing) LA to send priorities list to UKCA SMG
    1. UKCA API
    2. Review UKCA tests to cover all requirements
    3. Fix for c_h2o in Offline-Oxidants (bug #107)
    4. Updates/fixes to GLOMAP routines
    5. Standalone UKCA configurations
    6. Emissions flexibility
    7. Separare UKCA repository
    8. ACTIVATE optimisation
    9. FastJX-GLOMAP coupling
    10. Cloud-J
    11. Fixes for chemistry rates and ASAD (bugs #101 & #103)
    12. Rc fixes for water (bug #104)
    13. CH4 deposition fixes (bug #109)
    14. UKCA STASHmaster PP-codes
    15. Removal of redundant code
    16. GLOMAP-CLIM API refresh
    17. N48L85 configuration
    • Closed
  13. (new) Those with access check ukca_bugs regularly via OWA until Outlook access is fixed.
    • Ongoing
  14. (new) LA to take ownership of bug #107
    • Closed
  15. (new) LA to contact MH/OW regarding FastJX-GLOMAP coupling
    • Ongoing email discussions. Last from MH on 2019-06-19: "Thanks for the update on getting the FAST-JX GLOMAP coupling into the code. I will try my best to get it into the trunk at version 11.6. I might take you up on your kind offer to help with the tests when needed if that is ok?"
    • Ongoing - follow-up after UMvn11.5 is released.
  16. (new) MD to update um_ukca_eg app to be more in line with UKESM1, and add a rigorous test. An Age-of-air-only configuration being added due to other changes breaking this at vn11.4. Will still attempt to update the app if possible, otherwise bump to vn11.6.
    • Ongoing
  17. (new) LA to add N48 Linux and update EXP test
    • Ongoing
  18. (new) ALL to consider other rose-stem tests that could be included
    • There is a new GLOMAP-CLIM test. Plans for a UKCA-CASIM test.
  19. (new) LA to check-in with Rich Gilham after the Code Freeze regarding SC0138 keyword
    • Discussed at UMPB July 2019: General discussion if the SC0138 keyword is really necessary and if replacing it with an appropriate collab keyword. Final verdict: Time to stop using it (SC0138 keyword).
    • Closed
  20. (new) ALL to email LA with suggestions for next set of tutorials
    • Ongoing
  21. (new) LA to send Doodle poll for early September.

Standing actions

  1. (ongoing - standing item) CMG to monitor list of bugs and circulate between group for owners.
  2. (ongoing - standing item) At CMG future meetings there should be included an appendix listing the open bugs. If a ticket is included it is not an open bug (as the owner of ticket is person doing lodging).
  3. (ongoing - standing item) ALL with access to keep monitoring ukca_bugs email list.
  4. (ongoing - standing item) ALL to think about if code changes will change KGO and let systems team know in good time, and remember to include the UKCA & SC0138 keywords to all UKCA tickets. This should be a standard part of our working practice. Note also the required keywords.
  5. (ongoing - standing item) ALL to note, review, and remove obsolete/redundant code
  6. (ongoing - standing item) ALL closed ticket owners to produce a summary of their change for the UKCA newsletter for special occasions, e.g. to coincide with Composition-Climate Meeting.
  7. (ongoing - standing item) Code Owner to produce draft newsletter for CMG to approve.

Bugs/Bugfixes (LA)

Open Bugs

# Description Ticket Owner Comments Comments from this meeting
35 Error in labelling: Surface resistance <--> stomatal resistance ? UM:#1409 Fiona O'Connor Leave open. Garry Hayman started work in offline JULES.
Still ongoing. JM found bug in SO2 deposition, where there is an inconsistency when changing tiles. Making progress.
Leave open. GH gave update, has done a lot of work. Concerned it might not be bit-reproducable, but could be solvable. Encouraging to lodge existing. Resistances done as metadata. Not clear as to timescales.
FOC to encourage Garry Hayman to start lodging JULES code piecemeal.
Leave open
101 Errors in reaction rates and diagnostics UM:#4466 Luke Abraham / Scott Archer-Nicolls There is a need to conserve science config for UKESM1, so changes need to be on a flag.
Colin has been looking at DMSO, as this is not currently used in StratTrop but is in Offline.
Might be able to merge the rate with CEJ changes, but best to leave the diagnostic issues as a separate change.
LA, JM, CEJ, SAN to discuss offline and keep GM in loop.
Use versioning system. Talk about further offline.
Keep ongoing.
LA to do with #103
Do for vn11.6. Quite a large ticket.
103 Incorrect indexing of tmprc array: ASAD_SPIMPMJP UM:#4562 Scott Archer-Nicolls Breaks KGO - affects solver convergence. In practice doesn’t make a huge difference, but should go into vn11.4, but does change KGO. UKESM1 - small change might be OK - get in touch for advice - might be able to test with UKESM1 AMIP only. Check with Oliver Wild
Could be rolled-up with #101
LA to do with #101
Do with #101
104 Incorrect value for Rc for SO_2 deposition to open water UM:#4635 Catherine Hardacre Got to be separate to #35, unless there is an idea of timelines.
JM to find someone to progress bug #104.
LA to follow-up with JM
107 Missing c_h2o conversion factor in offline_oxidants_be um:#4714
um:#5118
Luke Abraham Code is zq(:)=RESHAPE(q(:,:,k),(/theta_field_size/)) rather than zq(:) = RESHAPE(q(:,:,k),(/theta_field_size/))/c_h2o
LA to take ownership and assess
108 Incorrect stash processing of GLOMAP load*, conc* and plume_scav diagnostics UM:#4909 Mohit Dalvi To be Lodged after 10.6
Do after CMIP6 runs are finished (after Jan 2020)
109 Bug in dry deposition of CH4 for 13/17/27 tiles UM:#4913 Gerd Folberth To be Lodged after 10.6
Gerd noted unusual CH4 output in CMIP6 runs.
May need a rose stem test for H2 and CH4 drdydep code lodged at UM:#4157
Makes a huge difference to CH4. Needs testing. AJH or GF will do during vn11.4, progress for vn11.6. May need UKESM1 approval.
Code is difficult to understand. AJH thinks GarryH is aware of this problem.

ACTION: LA to ask GF to add more details to ticket UM:#4913 to understand the problem (bug #109)

110 UKCA solar-cycle routine does not pass NRUN-CRUN test UM:#4828 Luke Abraham To be Lodged after 11.5 Have a solution.
111 Cloud fraction array (cloud_frac) passed with wrong number of levels
Array passed as (row_length, rows, :), so offset by 1 level under ENDGame
Part of UM:#4822 John Hemmings Affects all ENDGame versions. To be Lodged for 11.5 Do as separate ticket after #4822 committed. MD #5009 testing
112 delh2so4_chem is calculated incorrectly UM:#5076 Luke Abraham To be Lodged for 11.6.
Incorrectly "fixed" in UM:#3245 (Bug #87).
All StratTrop+GLOMAP versions post vn8.2. Made worse post-vn11.4 if logical l_fix_ukca_h2so4_ystore is True.
113 Latitude bugs affecting LAM configurations UM:#5119 John Hemmings To be lodged for 11.6 ACTION: ALL to consider thinking about fixing bug #113 (see UM:#5119)

Redundant Code (LA)

Code for deletion

PLEASE REMEMBER TO EDIT THE ORIGINAL REDUNDANT CODE TABLE AND NOT THIS PAGE

# Description Ticket Remarks Status Timeframe Owner Comments from previous meetings Comments from this meeting
2 Old dry deposition scheme UM:#2356 None PROPOSED Medium-term Nick Savage
4 ukca_strat_update.F90 - relax_ozone UM:#1505 None PROPOSED Medium-term Luke Abraham
5 Backward Euler Standard Tropospheric Chemistry UM:#1507 None PROPOSED Medium-term Fiona O'Connor
6 Wet oxidation of SO2 in GLOMAP UM:#4475 None PROPOSED Medium-term Colin Johnson JM to look at this re: UKESM1 c.f. offline GLOMAP. JM to take ownership. Still needed for CTM.

Relevance for standalone code and later coupling to CTM.

7 Updraft velocity - l_conv_tke UM:#3668 Proposed in December 2017. Approval being sought for GA series (likely GA8). PROPOSED Medium-term Alan J Hewitt / Ian Boutle
8 All backward Euler chemistry schemes None Note that this is only for the “hard-coded” BE schemes. Quite a lot of work would be involved, but good as a longer-term aim. Note that problems for AQUM would need to be addressed first. PROPOSED Long-term Luke Abraham
9 Deprecated UKCA netcdf attribute options UM:#2513 'emission_type', string type attributes no longer recommeneded. Need some time for most new code to adopt these. Still used in AQUM files. DEPRECATED Medium-term Nick Savage AQUM are addressing the files. Have gone from operational jobs, have ticket to remove from rose-stem. LN to document. These have been removed from AQUM.
AQUM work has been done.
From AS: In the ancil ticket linked from #2513 (https://code.metoffice.gov.uk/trac/ancil/ticket/782) it says that GA7 depends on the old format files, and as far as I know this is still true. In the 3-step timeline suggested, we haven't reached step 2 unfortunately. That ticket was about ensuring GA8 did not use the old attributes. I believe that GA8 is due for release in the next few months (it has been a very long time coming), so UM vn11.5 may be the last version at which GA7 is supported. Do you mind waiting a bit longer?
10 ASAD families chemistry None Families chemistry has not been used for many years (since at least UM6.1. It is likely that it no longer works if turned on, and it complicates the code making other developments harder DEPRECATED Short-term Luke Abraham
11 Dry deposition 9 PFT option None The 13 PFT option has been adopted for UKESM1.
The 9 PFT option is possibly not being used at all.
Proposed in June 2018.
PROPOSED Long-term Alan J Hewitt
12 Remove explicit C_SPECIES constants and calculate these from M_SPECIES and the mass of air None Will have wide-ranging rose-stem KGO breakages PROPOSED Short-term Luke Abraham
13 Remove temporary logical l_fix_nacl_density
Introduced at UM:#2405
UM:#3490 Requires AMIP testing and a GMED ticket
Could be accepted to GA9
PROPOSED Medium-term Jane Mulcahy See JM GMED action
15 Remove temporary logical l_fix_ukca_impscav
Introduced at UM:#1421
UM:#4145 Approval being sought for GA series (likely GA9)
GMED:#402
PROPOSED Medium-term Alan J Hewitt See JM GMED action
16 Remove temporary logical l_fix_improve_drydep
Introduced at UM:#1250
UM:#3997 Approval being sought for GA series (likely GA9)
GMED:#393
PROPOSED Medium-term Alan J Hewitt See JM GMED action
17 Remove temporary logical l_fix_ukca_h2dd_x
Introduced at UM:#4157
UM:#4245 Approval being sought for GA series (likely GA9)
GMED:#410
PROPOSED Medium-term Alan J Hewitt See JM GMED action
18 Remove temporary logical l_fix_neg_pvol_wat
Introduced at UM:#4383
UM:#4416 Approval being sought for GA series (likely GA9)
GMED:#421
PROPOSED Medium-term Alan J Hewitt
20 Remove MODULEs/SUBROUTINEs/FUNCTIONs
within UKCA not called within the UM:

MODULE asad_emissn_mod and SUBROUTINE asad_emissn
MODULE asad_inemit_mod and SUBROUTINE asad_inemit
UM:#4827 Requires removing additional redundant emission code from ukca_chem_master.F90 vn11.3_remove_asad_emissn_mod DEPRECATED Short-term Alan J Hewitt Should be finished for vn11.4 Done. Closed
21 Remove code behind L_use_stratclim from ukca_fastjx_mod
Introduced prior to MOSRS
UM:#4875 This could either be properly included via a namelist or removed from the UM PROPOSED Medium-term Alan J Hewitt LA contact OW about progressing FastJX-RADAER work
22 Remove temporary logical l_fix_ukca_input_meoh_emiss
Introduced at UM:#3031
UM:#4848 Approval being sought for GA series (likely GA9)
GMED:#461
PROPOSED Medium-term Mohit Dalvi

Update on UKCA API redesign

JH has been working on redesigning the UM-UKCA interface to remove the model specific coupling. His proposed design document can be found here:

JH: Focussing on structural tickets. Good progress made. Working on tracer handling and the environmental drivers ticket & trace gas mix ratios. STASH Tag98 still needed but now purely on the UM side. MD: Have been working on emissions. Emissions working without changing answers.

Progress with Priorities (LA)

The current priorities are:

  1. UKCA API
  2. Review UKCA tests to cover all requirements
  3. Fix for c_h2o in Offline-Oxidants (bug #107)
  4. Updates/fixes to GLOMAP routines
  5. Standalone UKCA configurations
  6. Emissions flexibility
  7. Separare UKCA repository
  8. ACTIVATE optimisation
  9. FastJX-GLOMAP coupling
  10. Cloud-J
  11. Fixes for chemistry rates and ASAD (bugs #101 & #103)
  12. Rc fixes for water (bug #104)
  13. CH4 deposition fixes (bug #109)
  14. UKCA STASHmaster PP-codes
  15. Removal of redundant code
  16. GLOMAP-CLIM API refresh
  17. N48L85 configuration

GM: Highlighted the HyDiS gas-particle partitioning module for nitrate aerosol (with ammonium and interactive pH), currently on a vn7.3 branch. AH: strategically nitrate is there, raise with AH and/or KC.

ACTION: GM to email AH/KC regarding nitrate

UKCA Testing Requirements (LA)

It would be helpful to fully document what is expected regarding code changes, e.g. rose-stem, AMIP & short tests, AQUM tests etc.

Initial details have been added here: Testing Requirements for UKCA Code Changes

ACTION: ALL to make changes to Testing Requirements for UKCA Code Changes wiki page as they see fit.

UM11.5 & UM11.6 Tickets

See here for UM vn11.5 tickets on MOSRS.

See here for UM vn11.6 tickets on MOSRS.

PLEASE REMEMBER TO USE THE UKCA KEYWORD FOR ALL UKCA CHANGES

UM vn11.5 is a CLOSED RELEASE for UKCA changes (except for API changes).

MR: Discussed inheriting Chris Dearden’s tickets. LA to talk offline

ACTION: MR & LA to talk offline regarding Chris Dearden's tickets.

AOB/DNM (LA)

AJH - planned refactoring work to get benefits of OpenMP (removing IF inside DO loops). See changes to ukca_mode_setup.F90.

ACTION: LA to contact AJH/GM offline regarding coupling StratTrop to GLOMAP-CLIM

LN: Will be going on maternity leave, and so will need a replacement AQUM representative for these meetings.

ACTION: LA to send message to Paul Agnew regarding an AQUM representative to these meetings

Date of next meeting (LA)

ACTION: LA to send Doodle poll for early November.