id	summary	reporter	owner	description	type	status	priority	milestone	component	version	resolution	keywords	cc
526	Missing T_PASSIVE	arango	arango	"The '''T_PASSIVE''' conditional is missing in get_data.F and set_data.F whe processing surface and bottom inert tracers fluxes. If there is not net surface or bottom flux for these tracers, the code it is still correct since '''stflx''' and '''btflx''' is initialized to zero when the array are allocated. We need to have the following logic when processing such fields:
{{{
# if defined BIOLOGY || defined SEDIMENT || defined T_PASSIVE
  ...
# endif
}}}
The '''TLM''', '''RPM''', '''ADM''' version of these files are also corrected. Many thanks to Lyon Lanerolle for [https://www.myroms.org/forum/viewtopic.php?f=14&t=2462 reporting] this problem.

I also corrected the '''units''' attribute in output NetCDF files for '''Wsed''', '''tau_ce''', and '''tau_cd'''. We need to have instead:

{{{
        double Wsed(NST) ;
                Wsed:long_name = ""sediment particle settling velocity"" ;
                Wsed:units = ""meter second-1"" ;
        double tau_ce(NST) ;
                tau_ce:long_name = ""sediment critical shear for erosion"" ;
                tau_ce:units = ""meter-2 second-2"" ;
        double tau_cd(NST) ;
                tau_cd:long_name = ""sediment critical shear for deposition"" ;
                tau_cd:units = ""meter-2 second-2"" ;
}}}
Many thanks to Changwei Bian for [https://www.myroms.org/forum/viewtopic.php?f=19&t=2460 bringing] this to my attention.
"	bug	closed	major	Release ROMS/TOMS 3.6	Nonlinear	3.6	Fixed