Difference between revisions of "Bugfixes (new approach)"
From UKCA
| Line 119: | Line 119: | ||
| 46 || Inconsistent use of ''wet_levels'' and ''model_levels'' <br/> in UKCA_AERO_CTL 1-D arrays || None || || All versions || None || '''Impact if wet_levels/= model_levels''' |
| 46 || Inconsistent use of ''wet_levels'' and ''model_levels'' <br/> in UKCA_AERO_CTL 1-D arrays || None || || All versions || None || '''Impact if wet_levels/= model_levels''' |
||
|- |
|- |
||
| − | | 47 || Dimension mismatch: q_chem, t_chem not <br /> considering extra level for ENDGame || [https://code.metoffice.gov.uk/trac/um/ticket/379 #379 (MOSRS)] || || ENDGame-UKCA || Example fix [https://code.metoffice.gov.uk/trac/um/changeset/2703/main/branches/dev/mohitdalvi/vn10.0_glomap_coagnucl_fix/src/atmosphere/UKCA/ukca_main1-ukca_main1.F90 here] || To be lodged at UM10.1 |
+ | | 47 || Dimension mismatch: q_chem, t_chem not <br /> considering extra level for ENDGame || [https://code.metoffice.gov.uk/trac/um/ticket/379 #379 (MOSRS)] || || ENDGame-UKCA || Example fix [https://code.metoffice.gov.uk/trac/um/changeset/2703/main/branches/dev/mohitdalvi/vn10.0_glomap_coagnucl_fix/src/atmosphere/UKCA/ukca_main1-ukca_main1.F90 here] (MOSRS) || To be lodged at UM10.1 |
|- |
|- |
||
| − | | 48 || Bugs in RADAER - dimensions, asymmetry param || [https://code.metoffice.gov.uk/trac/um/ticket/380 #380 (MOSRS)] || || All RADAER || Example fix [https://code.metoffice.gov.uk/trac/um/changeset/2651 here] and <br/> MO: ''fcm:um_br/dev/hadna/<br/>vn8.6_limit_asymmetry_parameter'' || To be lodged at UM10.1. Leads to <br/> div-by-zero error in rescale_phase_fnc |
+ | | 48 || Bugs in RADAER - dimensions, asymmetry param || [https://code.metoffice.gov.uk/trac/um/ticket/380 #380 (MOSRS)] || || All RADAER || Example fix [https://code.metoffice.gov.uk/trac/um/changeset/2651 here] (MOSRS) and <br/> MO: ''fcm:um_br/dev/hadna/<br/>vn8.6_limit_asymmetry_parameter'' || To be lodged at UM10.1. Leads to <br/> div-by-zero error in rescale_phase_fnc |
|- |
|- |
||
| − | | 49 || UKCA emissions: incorrect number, <br/>un-initialised slots in all_emissions() || [https://code.metoffice.gov.uk/trac/um/ticket/381 #381 (MOSRS)] || || All UKCA || Example fix [https://code.metoffice.gov.uk/trac/um/changeset?reponame=&new=3013%40main%2Fbranches%2Fdev%2Fmohitdalvi%2Fvn10.0_glomap_coagnucl_fix&old=2703%40main%2Fbranches%2Fdev%2Fmohitdalvi%2Fvn10.0_glomap_coagnucl_fix here] || To be lodged at UM10.1 |
+ | | 49 || UKCA emissions: incorrect number, <br/>un-initialised slots in all_emissions() || [https://code.metoffice.gov.uk/trac/um/ticket/381 #381 (MOSRS)] || || All UKCA || Example fix [https://code.metoffice.gov.uk/trac/um/changeset?reponame=&new=3013%40main%2Fbranches%2Fdev%2Fmohitdalvi%2Fvn10.0_glomap_coagnucl_fix&old=2703%40main%2Fbranches%2Fdev%2Fmohitdalvi%2Fvn10.0_glomap_coagnucl_fix here] (MOSRS) || To be lodged at UM10.1 |
|- |
|- |
||
| − | | 50 || ukca_solve_coagnucl failures || [https://code.metoffice.gov.uk/trac/um/ticket/319 #319 (MOSRS)] || || UKCA-GLOMAP || Example fix [https://code.metoffice.gov.uk/trac/um/changeset/2582/main/branches/dev/mohitdalvi/vn10.0_glomap_coagnucl_fix here] || '' Mainly a workaround'' <br/> To be lodged at UM10.1 |
+ | | 50 || ukca_solve_coagnucl failures || [https://code.metoffice.gov.uk/trac/um/ticket/319 #319 (MOSRS)] || || UKCA-GLOMAP || Example fix [https://code.metoffice.gov.uk/trac/um/changeset/2582/main/branches/dev/mohitdalvi/vn10.0_glomap_coagnucl_fix here] (MOSRS) || '' Mainly a workaround'' <br/> To be lodged at UM10.1 |
|- |
|- |
||
|} |
|} |
||
Revision as of 17:30, 9 March 2015
The aim of this page is to present the information on the new bug reporting system. All bugs should be reported to ukca_bugs@metoffice.gov.uk. More details of the approach to reporting and fixing bugs is contained in the document here .
Bug fixes table
This is a list of all reported bugs, with a link to tickets with more information on the issue and branches to fix it. Some of the bugs/ issues are described in more detail on an older bugfixes page. The UM versions listed are those where the bug is known to be present. Please note however, that bugs in versions before 8.2 are not reported here, although information on some of these can be found here.
| # | Description | Ticket | Affected Versions/ Config |
Branch with fix | Remarks | |
|---|---|---|---|---|---|---|
| MetO | NCAS | |||||
| 1 | Improper construction of filename for Background Aerosol data used for Fast-JX/ HetPSC |
#4283 | #439 | 8.2-8.3 | MetO: fcm:um_br/pkg/Share/vn8.2_pkg_ukca_bugfixes Puma: fcm:um_br/dev/nhsavage/vn8.2_pkg_ukca_bugfixes |
Lodged at UM8.3. Note: Likely to clash with some vn8.2 science branches. |
| 2 | SO3 missing from UKCA_CSPECIES | |||||
| 3 | MODE diagnostics indexing issue | |||||
| 4 | Correct the methane initialisation when l_prescribe_ch4 option is selected | |||||
| 5 | Logic for Heterogenous reactions under TropIsop Chem | |||||
| 6 | String length differences in Strat+Trop reaction rate definition array | |||||
| 7 | Incorrect Loop control for a Fast-JX routine | |||||
| 8 | Remove N2O from lumped N species for Stratospheric Chemistry | |||||
| 9 | Fix an initialisation issue that caused consecutive runs to give different results | |||||
| 10 | Incorrect read-size for a Fast-JX lookup table | |||||
| 11 | Minor write statement formatting issues | |||||
| 12 | Minor variable/ array initialisation issues. | |||||
| 13 | Missing initialisation of COS LowBC | #4956 | #651 | 8.2 CheST+Aero |
MetO: fcm:um_br/dev/hadfo/vn8.2_ukca_COSmmr_bugfix Puma: fcm:um_br/dev/foconnor/vn8.2_ukca_COSmmr_bugfix |
Lodged at UM8.2 |
| 14 | UKCA Age-of-Air setup bug | #5175 | #636 | 8.2-8.5 AgeAirOnly |
MetO: fcm:um_br/dev/hadzm/vn8.5_ukca_AgeAirOnly_fix Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.5 |
| 15 | Incorrect factors for MODE emiss diagnostics | Part of #4301 |
#636 | 8.4-8.5 GLOMAP |
MetO: Included in fcm:um_br/dev/hadcj/vn8.4_ukca_combined_scav fcm:um_br/dev/hadcj/vn8.5_ukca_combined_scav Puma: fcm:um_br/dev/cce_cj/vn8.4_ukca_combined_scav fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.5 |
| 16 | Array Dimensioning/ Indexing issues affecting ENDGAME |
#5420 | #636 | 8.4-8.5 EG+UKCA |
MetO: fcm:um_br/dev/hadzm/vn8.5_ukca_dims_uninit_fixes Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.5 |
| 17 | Activate: Value underflow trap combined with debug-level | Part of #5420 |
#636 | 8.2-8.5 Activate |
MetO: Included in fcm:um_br/dev/hadzm/vn8.5_ukca_dims_uninit_fixes Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.5 |
| 18 | Errors in UKCA CheST+Het/PSC setup code | #5465 | #652 | 8.5 CheST+ Het/PSC |
MetO: fcm:um_br/dev/fris/vn8.5_fix_het_psc Puma: fcm:um_br/dev/foconnor/vn8.5_fix_het_psc |
Lodged at UM8.5 |
| 19 | Change in UKCA results when print/ debug level is changed |
#5470 | #636 | 8.2-8.6 GLOMAP |
MetO: fcm:um_br/dev/hadzm/vn8.6_ukca_fix_prstatus_diff Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.6 |
| 20 | Failures in asad_chem_diags when Print-level=Diag and using UmPrint as Asad debug output exceeds buffer size |
Part of #5470 | #650 | 8.6 | MetO: Included in fcm:um_br/dev/hadzm/vn8.6_ukca_fix_prstatus_diff Puma: fcm:um_br/dev/mdalvi/vn8.6_ukca_remove_asad_prints |
Lodged at UM8.6 |
| 21 | Incorrect calculation of some ASAD 3-D diags | #5922 | #636 | 8.2-8.6 | MetO: fcm:um_br/dev/hadlk/vn8.6_UKCA_3Demis_FixUnits Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.6 |
| 22 | Indexing bug - sin_v_latitude | Part of #5420 |
#636 | 8.5 EG+UKCA |
MetO: Included in fcm:um_br/dev/hadzm/vn8.5_ukca_dims_uninit_fixes Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at UM8.5 |
| 23 | Bug in Fast-JX mass calculation | #5404 | #565 Also #636 |
7.3: FastJX |
MetO: fcm:um_br/dev/frtf/vn8.4_fastj-x_debug Puma: fcm:um_br/dev/luke/vn8.4_FJX_dm_3d_bugfix fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Seems to mostly affect Low-top or hi-res runs. Lodged at vn9.0 |
| 24 | Erroneous 50% reduction factor in aerosol sedimentation velocity calc |
Part of #6105 | Part of #647 | 8.2: GLOMAP |
MetO:Included in fcm:um_br/dev/frjy/vn8.4_ukca_vgrav_av fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/dan2012/vn8.4_Sedimentation_Fix fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 25 | ACTIVATE uses saturation mix ratio instead of saturation vap pressure |
Part of #6105 | #636 | 8.2: ACTIVATE |
MetO: fcm:um_br/dev/frjy/vn8.4_ukca_vgrav_av fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: Included in fcm:um_br/dev/dan2012/vn8.4_Nenes_Activate fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 26 | TropChem: uses mmr instead of vmr for Q | Part of #6105 | #636 | 8.2: TropIsop |
MetO: fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 27 | Erroneous reaction rate for OH+MGLY reaction | Part of #6105 | #636 | 8.2: | MetO: For StratTrop/CheST fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 for TropIsop Included in: fcm:um_br/dev/hadfo/vn8.4_ukca_ update_tropisop_rate_coeffs Puma: For StratTrop/CheST fcm:um_br/dev/foconnor/vn8.4_ukca_bugfix _StratTrop_OHMGLY_rxnrate For TropIsop/CheT: Included in fcm:um_br/dev/foconnor/vn8.4_vn8.4_ukca _update_tropisop_rate_coeffs fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 (StratTrop only) |
| 28 | Indexing bug - all_tracers array in Plume Scavenging branch |
Part of #6105 | None | 8.4: EG+Plume Scav |
MetO: Included in fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 29 | Indexing bug - asad_chemdiags%diag_type in asad_tendency_ste | Part of #6105 | #636 | 8.2: | MetO: Included in fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 30 | Indexing bug - f3_at_u | Part of #6105 | #636 | 8.4: EG+UKCA |
MetO: Included in fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 31 | FP exceptions in asad_sparse_vars or asad_spimjmp |
#5616, #368 (MOSRS) | #555 | 7.3: CheT+Aero |
MetO: fcm:um_br/dev/fris/vn8.4_increase_rafeps Puma: fcm:um_br/dev/nhsavage/vn8.4_increase_rafeps |
Temporary Work-around. May not work in all cases |
| 32 | ACTIVATE:uses cloud_liq_water (diagnostic) which is not consistent with cloud fraction (prognostic) |
Part of #6105 | #636 | 7.3: ACTIVATE |
MetO: fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 33 | RADAERv2 in UMUI: Look-up table files linked to wrong Unit numbers | None | None | 8.5-8.6 | Enter _an_ files for accumulation mode and _ac_ files for the aitken/insol accum mode boxes on the RADAER panel |
No code change involved |
| 34 | ACTIVATE: Argument dimensions not compatible with ENDGame (should specify 1:model_levels) |
Part of #6105 | #636 | 8.5: EG-Activate |
MetO: Included in fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_GA7 Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
Lodged at vn9.0 |
| 35 | Error in labelling: Surface resistance <--> stomatal resistance ? | #6417 | #653 | 7.3: | None | Awaiting input from Federico Centoni |
| 36 | Error in calculation of Column Ozone diagnostic | #102 | 7.3: | MetO: fcm:um_br/dev/hadvh/vn8.2_coloz_fix Puma: fcm:um_br/dev/foconnor/vn8.2_coloz_fix |
To be lodged at vn10.2 | |
| 37 | Mismatch in levels for UKCA non-transported prognostics/ chem_diags. Dimensions not compatible with ENDGame |
#6376 | #649 | 9.1 ENDGame-UKCA |
MetO: fcm:um_br/dev/hadzm/vn9,1_ukca_fix_chemdiag_levs Puma: fcm:um_br/dev/mdalvi/vn9.1_ukca_fix_chemdiag_levs |
Lodged at vn9.1 |
| 38 | Missing logical for volcanic so2 emissions. | #6418 | #656 | MetO: fcm:um_br/dev/hadna/vn8.6_deactivate_volcanoes Puma: None |
To be Lodged | |
| 39 | Fast-JX fails if .NOT. L_PC2 and .NOT. l_3d_cca | #6443 | #667 | 8.2-9.1 | MetO: fcm:um_br/dev/fris/vn9.1_fastjx_index_bug Puma: fcm:um_br/dev/nhsavage/vn9.1_fastjx_index_bug |
Lodged at vn9.1 |
| 40 | UKCA fails if MODE switched Off 10m wind not in D1 but req for AeroChem |
Part of #112 |
9.1 UKCA+AeroChem; MODE & CLASSIC Off |
MOSRS: fcm:um.xm_br/dev/mohitdalvi/vn10.0_ukca_update_app |
Lodged at vn10.0 | |
| 41 | OC biomass burning emissions - incorrect conversion. Assumes emissions as kg POM, but (new) ancils are kg C |
#6429 | vn8.5: | MetO: fcm:um_br/dev/fpbj/vn8.6_ukca_mode_BBOC-POM_bugfix Puma: None |
To be lodged Need to determine which ancils are POM vs C | |
| 42 | RADAER lookup table reading bug. Input values not passed to RADAER |
#6616 part of MOSRS #112 |
vn9.2: with RADAER |
MetO: fcm:um_br/dev/hadzm/vn9.2_ukca_fix_radnml_read MOSRS: fcm:um.xm_br/dev/mohitdalvi/vn10.0_ukca_update_app |
Lodged at vn10.0 | |
| 43 | Reconfiguration bug. L_UKCA used for setting tr_levels before RUN_UKCA is read |
vn8.3:vn9.0 UKCA with Classic fully Off |
MetO: fcm:um_br/dev/hadzm/vn8.6_fix_rcf_stshcomp_ukca Puma: None |
Automatically fixed under STSHCOMP rationalisation at vn8.6 (#5331) | ||
| 44 | Logicals set to default TRUE: l_ukca_ibvoc: aborts if JULES copy is not set. l_ukca_plume_scav: non-GLOMAP (e.g. Age-Air only) run crashes. |
part of MOSRS #112 | vn9.2: UKCA | MOSRS: fcm:um.xm_br/dev/mohitdalvi/vn10.0_ukca_update_app | Lodged at vn10.0 | |
| 45 | nlev is not passed to asad_impact in asad_cdrive | #252 (MOSRS) | All versions | MOSRS: fcm:um.x_br/dev/scottwales/r1974_asad_impact_argument |
See MOSRS ticket #252. Lodged in vn10.1. EXBUG from NCI. | |
| 46 | Inconsistent use of wet_levels and model_levels in UKCA_AERO_CTL 1-D arrays |
None | All versions | None | Impact if wet_levels/= model_levels | |
| 47 | Dimension mismatch: q_chem, t_chem not considering extra level for ENDGame |
#379 (MOSRS) | ENDGame-UKCA | Example fix here (MOSRS) | To be lodged at UM10.1 | |
| 48 | Bugs in RADAER - dimensions, asymmetry param | #380 (MOSRS) | All RADAER | Example fix here (MOSRS) and MO: fcm:um_br/dev/hadna/ vn8.6_limit_asymmetry_parameter |
To be lodged at UM10.1. Leads to div-by-zero error in rescale_phase_fnc | |
| 49 | UKCA emissions: incorrect number, un-initialised slots in all_emissions() |
#381 (MOSRS) | All UKCA | Example fix here (MOSRS) | To be lodged at UM10.1 | |
| 50 | ukca_solve_coagnucl failures | #319 (MOSRS) | UKCA-GLOMAP | Example fix here (MOSRS) | Mainly a workaround To be lodged at UM10.1 | |
Package branches table
For every supported version, the aim will be to have a package that fixes all bugs known to exist at that UM version. The package will be supported on both the NCAS and Met Office repositories (until a single repository is created)
| UM Versions | Ticket | Package Branch | |
|---|---|---|---|
| MetO | NCAS | ||
| 8.2 | #4382 | #439 | Meto: fcm:um_br/pkg/Share/vn8.2_pkg_ukca_bugfixes Puma: fcm:um_br/dev/nhsavage/vn8.2_pkg_ukca_bugfixes |
| 8.3 | #1 | #2 | |
| 8.4 | #1 | #636 | Puma: fcm:um_br/dev/mdalvi/vn8.4_ukca_fixes_from_vn86 |
| 8.6 | #6105 and #5404 | #647 | Meto: fcm:um_br/dev/hadfo/vn8.6_ukca_bugfixes_package_for_release_jobs Puma: fcm:um_br/dev/foconnor/vn8.6_ukca_bugfixes_package_for_release_jobs |
