From 99fb8bffdb619b24af63228dc6ff33472657a839 Mon Sep 17 00:00:00 2001
From: Kay Robbins <1189050+VisLab@users.noreply.github.com>
Date: Thu, 18 Apr 2024 15:36:07 -0500
Subject: [PATCH] Corrected argument on summarize column values

---
 hed/tools/remodeling/operations/summarize_column_values_op.py | 4 ++--
 .../remodeling/operations/summarize_hed_validation_op.py      | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/hed/tools/remodeling/operations/summarize_column_values_op.py b/hed/tools/remodeling/operations/summarize_column_values_op.py
index 3675e31f..0e8820cb 100644
--- 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
index 4b950221..77281aa4 100644
--- 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.