English and French metadata overhaul

HISTORY.rst

@@ -15,14 +15,16 @@ New features and enhancements
 Breaking changes
 ^^^^^^^^^^^^^^^^
 * `scipy` has been temporarily pinned below version 1.9 until lmoments3 tests can be rewritten to account for the new API. (:issue:`1142`, :pull:`1143`).
+* English indicator metadata has been adjusted to remove frequencies and other variable-sourced fields in the `long_name` of indicators. English indicators now have an explicit `title` and `abstract`. (:issue:`936`, :pull:`1123`).
+* French indicator metadata translations are now more uniform and follow agreed-upon grammar conventions, removing variable-sourced fields in `long_name_fr`. (:issue:`936`, :pull:`1123`).
 
 Bug fixes
 ^^^^^^^^^
 * Fixed ``saturation_vapor_pressure`` for temperatures in other units than Kelvins (also fixes ``relative_humidity_from_dewpoint``). (:issue:`1125`, :pull:`1127`).
 * Indicators that do not care about the input frequency of the data will not check the cell methods of their inputs. (:pull:`1128`).
 * Fixed the signature and docstring of ``heat_index`` by changing ``tasmax`` to ``tas``. (:issue:`1126`, :pull:`1128`).
-* Fixed a formatting issue with virtual indicator modules (`_gen_returns_section`) that was creating malformed `Returns` sections in `sphinx`-generated documentation. (:pull:`1131`).
 * Removed some artefact reference roles introduced in :pull:`1131` that were causing LaTeX builds of the documentation to fail. (:issue:`1154`, :pull:`1156`).
+* Fixed a formatting issue with virtual indicator modules (`_gen_returns_section`) that was creating malformed `Returns` sections in `sphinx`-generated documentation. (:pull:`1131`).
 
 Internal changes
 ^^^^^^^^^^^^^^^^

xclim/core/formatting.py

@@ -20,18 +20,23 @@
 from xclim.core.utils import InputKind, PercentileDataArray
 
 DEFAULT_FORMAT_PARAMS = {
-    "tasmin_per_thresh": "{unkown}",
-    "tasmin_per_window": "{unkown}",
-    "tasmin_per_period": "{unkown}",
-    "tas_per_thresh": "{unkown}",
-    "tas_per_window": "{unkown}",
-    "tas_per_period": "{unkown}",
-    "tasmax_per_thresh": "{unkown}",
-    "tasmax_per_window": "{unkown}",
-    "tasmax_per_period": "{unkown}",
-    "pr_per_thresh": "{unkown}",
-    "pr_per_window": "{unkown}",
-    "pr_per_period": "{unkown}",
+    "tasmin_per_thresh": "{unknown}",
+    "tasmin_per_window": "{unknown}",
+    "tasmin_per_period": "{unknown}",
+    "tas_per_thresh": "{unknown}",
+    "tas_per_window": "{unknown}",
+    "tas_per_period": "{unknown}",
+    "tasmax_per_thresh": "{unknown}",
+    "tasmax_per_window": "{unknown}",
+    "tasmax_per_period": "{unknown}",
+    "pr_per_thresh": "{unknown}",
+    "pr_per_window": "{unknown}",
+    "pr_per_period": "{unknown}",
+    # FIXME: The following params can be found in indices._multivariate
+    "tasmax_per": "{unknown}",
+    "tasmin_per": "{unknown}",
+    "tas_per": "{unknown}",
+    "pr_per": "{unknown}",
 }
 
 
@@ -53,7 +58,8 @@ def __init__(
         mapping : Mapping[str, Sequence[str]]
             A mapping from values to their possible variations.
         modifiers : Sequence[str]
-            The list of modifiers, must be the as long as the longest value of `mapping`. Cannot include reserved modifier 'r'.
+            The list of modifiers, must be the as long as the longest value of `mapping`.
+            Cannot include reserved modifier 'r'.
         """
         super().__init__()
         if "r" in modifiers:
@@ -427,8 +433,7 @@ def _call_and_add_history(*args, **kwargs):
 def gen_call_string(funcname: str, *args, **kwargs):
     """Generate a signature string for use in the history attribute.
 
-    DataArrays and Dataset are replaced with their name, while Nones, floats,
-    ints and strings are printed directly.
+    DataArrays and Dataset are replaced with their name, while Nones, floats, ints and strings are printed directly.
     All other objects have their type printed between < >.
 
     Arguments given through positional arguments are printed positionnally and those