#886: Clarify behaviour in docstring.

Author: mo-gill

mip_convert/mip_convert/requested_variables.py

@@ -139,8 +139,7 @@ def produce_mip_requested_variable(
         # cell_measures attribute via cmor.set_variable_attribute).
         cmor_variable = create_cmor_variable(time_slice)
         if not cell_measures_applied:
-            if saver.varid is None:
-                saver._getVarId(cmor_variable)
+            saver._getVarId(cmor_variable)
             # Apply cell_measures if a <mip_era>_cell_measures.json file exists.
             saver.cmor.apply_cell_measures(
                 user_config.mip_era,

mip_convert/mip_convert/save/cmor/cmor_wrapper.py

@@ -120,7 +120,13 @@ def set_frequency(self, frequency, **kwargs):
 
     def apply_cell_measures(self, mip_era, mip_table_dir, realm, variable, frequency, region, variable_id):
         """
-        Add cell measures read from json file in with mip tables
+        Set the cell_measures attribute on a CMOR variable using a lookup in
+        ``{mip_era}_cell_measures.json`` from the MIP tables directory. Returns
+        silently if the file does not exist.
+
+        The lookup key has the form ``{realm}.{root_label}.{branding}.{frequency}.{region}``
+        (variable name split on ``_``), so entries in the JSON must follow the
+        convention used for CMIP7.
         """
         self._debug_on_args('apply_cell_measures', [mip_era, mip_table_dir, realm, variable, frequency, region, variable_id], {})