SDATE_GFS in the context of metp is the first date that METP was run …

…and started generating statistics. This is different when doing cycled (gfs_cyc determines SDATE_GFS). In the case of forecast-only, SDATE_GFS is always the first free-forecast

workflow/rocoto/gfs_tasks.py

@@ -890,9 +890,11 @@ def metp(self):
         dependencies = rocoto.create_dependency(dep_condition='and', dep=deps)
 
         metpenvars = self.envars.copy()
-        metpenvar_dict = {'SDATE_GFS': self._base.get('SDATE_GFS'),
-                          # TODO - in Forecast-only, this is `SDATE` on the RHS
-                          'METPCASE': '#metpcase#'}
+        if self.app_config.mode in ['cycled']:
+            metpenvar_dict = {'SDATE_GFS': self._base.get('SDATE_GFS').strftime("%Y%m%d%H")}
+        elif self.app_config.mode in ['forecast-only']:
+            metpenvar_dict = {'SDATE_GFS': self._base.get('SDATE').strftime("%Y%m%d%H")}
+        metpenvar_dict['METPCASE'] = '#metpcase#'
         for key, value in metpenvar_dict.items():
             metpenvars.append(rocoto.create_envar(name=key, value=str(value)))