Fix warnings

Author: ufechner7

ext/VortexStepMethodControlPlotsExt.jl

@@ -62,7 +62,10 @@ function VortexStepMethod.save_plot(fig, save_path, title; data_type=".pdf")
     @debug "Current working directory: $(pwd())"
 
     try
-        fig.savefig(full_path)
+        hasproperty(fig, :savefig) || throw(ArgumentError(
+            "Figure object of type $(typeof(fig)) does not support savefig()."
+        ))
+        getproperty(fig, :savefig)(full_path)
         @debug "Figure saved as $data_type"
 
         if isfile(full_path)
@@ -658,20 +661,29 @@ function VortexStepMethod.plot_polars(
     # Load literature data if provided
     if !isempty(literature_path_list)
         for path in literature_path_list
-            data = readdlm(path, ',')
-            header = lowercase.(string.(data[1, :]))
+            raw_data = readdlm(path, ',')
+            table, header = if raw_data isa Tuple
+                # readdlm(...; header=true) returns (data, header)
+                raw_table, raw_header = raw_data
+                raw_table, lowercase.(string.(vec(raw_header)))
+            else
+                # Header is in first row when a single matrix is returned
+                raw_data[2:end, :], lowercase.(string.(raw_data[1, :]))
+            end
             # Find column indices for alpha, CL, CD, CS (case-insensitive, allow common variants)
             alpha_idx = findfirst(x -> occursin("alpha", x) || x == "aoa", header)
             cl_idx    = findfirst(x -> occursin("cl", x), header)
             cd_idx    = findfirst(x -> occursin("cd", x), header)
             cs_idx    = findfirst(x -> occursin("cs", x), header)
+            (isnothing(alpha_idx) || isnothing(cl_idx) || isnothing(cd_idx)) &&
+                throw(ArgumentError("Literature CSV must contain alpha/aoa, cl and cd columns: $path"))
             # Fallback: if CS not found, fill with zeros
-            cs_col = cs_idx === nothing ? zeros(size(data, 1)-1) : data[2:end, cs_idx]
+            cs_col = cs_idx === nothing ? zeros(size(table, 1)) : table[:, cs_idx]
             # Push as [alpha, CL, CD, CS]
             push!(polar_data_list, [
-                data[2:end, alpha_idx],
-                data[2:end, cl_idx],
-                data[2:end, cd_idx],
+                table[:, alpha_idx],
+                table[:, cl_idx],
+                table[:, cd_idx],
                 cs_col
             ])
         end

ext/VortexStepMethodMakieExt.jl

@@ -249,7 +249,7 @@ Save a Makie figure to a file.
 # Keyword arguments
 - `data_type`: File extension (default: ".png", also supports ".jpeg")
 """
-function VortexStepMethod.save_plot(fig, save_path, title; data_type=".png")
+function VortexStepMethod.save_plot(fig::Makie.Figure, save_path, title; data_type=".png")
     isnothing(save_path) && throw(ArgumentError("save_path should be provided"))
 
     !isdir(save_path) && mkpath(save_path)
@@ -286,7 +286,7 @@ Display a Makie figure.
 # Keyword arguments
 - `dpi`: Dots per inch for the figure (default: 130) - currently unused in Makie
 """
-function VortexStepMethod.show_plot(fig; dpi=130)
+function VortexStepMethod.show_plot(fig::Makie.Figure; dpi=130)
     display(fig)
 end