Merge pull request #908 from VisLab/develop

Corrected argument on summarize column values
hed-standard · Apr 18, 2024 · 095a713 · 095a713
2 parents d63872c + 99fb8bf
commit 095a713
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/hed/tools/remodeling/operations/summarize_column_values_op.py b/hed/tools/remodeling/operations/summarize_column_values_op.py
@@ -90,8 +90,8 @@ def __init__(self, parameters):
         self.summary_filename = parameters['summary_filename']
         self.append_timecode = parameters.get('append_timecode', False)
         self.max_categorical = parameters.get('max_categorical', float('inf'))
-        self.skip_columns = parameters['skip_columns']
-        self.value_columns = parameters['value_columns']
+        self.skip_columns = parameters.get('skip_columns', [])
+        self.value_columns = parameters.get('value_columns', [])
         self.values_per_line = parameters.get('values_per_line', self.VALUES_PER_LINE)
 
     def do_op(self, dispatcher, df, name, sidecar=None):

diff --git a/hed/tools/remodeling/operations/summarize_hed_validation_op.py b/hed/tools/remodeling/operations/summarize_hed_validation_op.py
@@ -65,7 +65,7 @@ def __init__(self, parameters):
         self.summary_name = parameters['summary_name']
         self.summary_filename = parameters['summary_filename']
         self.append_timecode = parameters.get('append_timecode', False)
-        self.check_for_warnings = parameters.get('check_for_warnings', False)
+        self.check_for_warnings = parameters['check_for_warnings']
 
     def do_op(self, dispatcher, df, name, sidecar=None):
         """ Validate the dataframe with the accompanying sidecar, if any.