From 598c3dc2442db6676c31722df63f01c1ffa3072c Mon Sep 17 00:00:00 2001 From: Matt Fisher Date: Fri, 23 Aug 2024 11:37:51 -0600 Subject: [PATCH] Clarify some comments / docstrings --- icepyx/core/APIformatting.py | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/icepyx/core/APIformatting.py b/icepyx/core/APIformatting.py index 90b7f3ac6..0e7dc5f82 100644 --- a/icepyx/core/APIformatting.py +++ b/icepyx/core/APIformatting.py @@ -122,6 +122,9 @@ def combine_params(*param_dicts): """ Combine multiple dictionaries into one. + Merging is performed in sequence using `dict.update()`; dictionaries later in the + list overwrite those earlier. + Parameters ---------- params : dictionaries @@ -129,7 +132,7 @@ def combine_params(*param_dicts): Returns ------- - single dictionary of all input dictionaries combined + A single dictionary of all input dictionaries combined Examples -------- @@ -411,7 +414,10 @@ def build_params(self, **kwargs): for key in opt_keys: if key == "Coverage" and key in kwargs: - # DevGoal: make there be an option along the lines of Coverage=default, which will get the default variables for that product without the user having to input is2obj.build_wanted_wanted_var_list as their input value for using the Coverage kwarg + # DevGoal: make an option along the lines of Coverage=default, + # which will get the default variables for that product without + # the user having to input is2obj.build_wanted_wanted_var_list + # as their input value for using the Coverage kwarg self._fmted_keys.update( {key: _fmt_var_subset_list(kwargs[key])} )