write inputfile debug gui button added.

Author: Sierd

aeolis/gui/application.py

@@ -210,10 +210,19 @@ def create_input_file_tab(self, tab_control):
                                        command=self.browse_save_location)
         save_browse_button.grid(row=3, column=2, sticky=W, pady=5, padx=5)
 
-        # Save button
+        # Save button (diffs only)
         save_config_button = ttk.Button(file_ops_frame, text="Save Configuration", 
-                                       command=self.save_config_file)
+                           command=self.save_config_file)
         save_config_button.grid(row=4, column=1, sticky=W, pady=10, padx=10)
+        save_config_desc = ttk.Label(file_ops_frame, text="Writes only parameters that differ from defaults.")
+        save_config_desc.grid(row=4, column=2, sticky=W, pady=10, padx=5)
+
+        # Save full button (all params)
+        save_full_config_button = ttk.Button(file_ops_frame, text="Save Full Configuration", 
+                            command=self.save_full_config_file)
+        save_full_config_button.grid(row=5, column=1, sticky=W, pady=5, padx=10)
+        save_full_config_desc = ttk.Label(file_ops_frame, text="Writes every parameter, including defaults.")
+        save_full_config_desc.grid(row=5, column=2, sticky=W, pady=5, padx=5)
 
     def create_domain_tab(self, tab_control):
         # Create the 'Domain' tab
@@ -460,6 +469,31 @@ def save_config_file(self):
             messagebox.showerror("Error", error_msg)
             print(error_msg)
 
+    def save_full_config_file(self):
+        """Save the full configuration (including defaults) to a file"""
+        save_path = self.save_config_entry.get()
+        
+        if not save_path:
+            messagebox.showwarning("Warning", "Please specify a file path to save the configuration.")
+            return
+        
+        try:
+            # Update dictionary with current entry values
+            for field, entry in self.entries.items():
+                value = entry.get()
+                self.dic[field] = None if value.strip() == '' else value
+            
+            # Write the full configuration file
+            aeolis.inout.write_configfile(save_path, self.dic, include_defaults=True)
+            
+            messagebox.showinfo("Success", f"Full configuration saved to:\n{save_path}")
+            
+        except Exception as e:
+            import traceback
+            error_msg = f"Failed to save full config file: {str(e)}\n\n{traceback.format_exc()}"
+            messagebox.showerror("Error", error_msg)
+            print(error_msg)
+
     def toggle_color_limits(self):
         """Enable or disable colorbar limit entries based on auto limits checkbox"""
         if self.auto_limits_var.get():

aeolis/hydro.py

@@ -67,7 +67,7 @@ def interpolate(s, p, t):
     if p['process_tide']:
         # Check if SWL or zs are not provided by some external model
         # In that case, skip initialization
-        if not p['external_vars'] or('zs' not in p['external_vars']) :
+        if not p['external_vars'] or ('zs' not in p['external_vars']) :
 
             if p['tide_file'] is not None:
                 s['SWL'][:,:] = interp_circular(t,
@@ -159,7 +159,7 @@ def interpolate(s, p, t):
 
     if p['process_runup']:
         ny = p['ny']
-        if ('Hs' in p['external_vars']):
+        if p['external_vars'] and ('Hs' in p['external_vars']):
 
             eta, sigma_s, R = calc_runup_stockdon(s['Hs'], s['Tp'], p['beach_slope'])
             s['R'][:] = R

aeolis/inout.py

@@ -121,7 +121,7 @@ def read_configfile(configfile, parse_files=True, load_defaults=True):
     return p
 
 
-def write_configfile(configfile, p=None):
+def write_configfile(configfile, p=None, include_defaults=False):
     '''Write model configuration file
 
     Writes model configuration to file. If no model configuration is
@@ -138,6 +138,9 @@ def write_configfile(configfile, p=None):
         Model configuration file
     p : dict, optional
         Dictionary with model configuration parameters
+    include_defaults : bool, optional
+        If True, write all parameters including defaults; if False, skip
+        parameters equal to the default values
 
     Returns
     -------
@@ -154,6 +157,22 @@ def write_configfile(configfile, p=None):
     if p is None:
         p = DEFAULT_CONFIG.copy()
 
+    # Helper: safely determine if a value equals the default without broadcasting errors
+    def _is_default_value(key, value):
+        if key not in DEFAULT_CONFIG:
+            return False
+
+        default = DEFAULT_CONFIG[key]
+
+        try:
+            return np.array_equal(np.asarray(value, dtype=object),
+                                  np.asarray(default, dtype=object))
+        except Exception:
+            try:
+                return value == default
+            except Exception:
+                return False
+
     # Parse constants.py to extract section headers, order, and comments
     import aeolis.constants
     constants_file = aeolis.constants.__file__
@@ -239,8 +258,8 @@ def write_configfile(configfile, p=None):
             for key in section_keys:
                 value = p[key]
 
-                # Skip this key if its value matches the default
-                if key in DEFAULT_CONFIG and np.all(value == DEFAULT_CONFIG[key]) :
+                # Skip this key if its value matches the default and skipping is allowed
+                if not include_defaults and _is_default_value(key, value):
                     continue
 
                 comment = comments.get(key, '')
@@ -263,8 +282,8 @@ def write_configfile(configfile, p=None):
             for key in sorted(remaining_keys):
                 value = p[key]
 
-                # Skip this key if its value matches the default
-                if key in DEFAULT_CONFIG and np.all(value == DEFAULT_CONFIG[key]):
+                # Skip this key if its value matches the default and skipping is allowed
+                if not include_defaults and _is_default_value(key, value):
                     continue
 
                 comment = comments.get(key, '')