EASY passing linter checks

workflow/templates/easy/NImParamRadiusGui.m

@@ -21,6 +21,7 @@ function NImParamRadiusGui(scansDir)
   global numCols
   global scanSize
   global scanMax
+  global searchRangeFile
 
   % Ncode ImRobot adaptation
   % TODO this code block and variables needs explanation
@@ -38,17 +39,17 @@ function NImParamRadiusGui(scansDir)
   selScan=1;
   SWgrowthArea=1;
 
-  if exist(pointMapsFile)
+  if exist(pointMapsFile, 'file')
     load(pointMapsFile);
   else
     load(fullfile(PTmats,'NImParameters')) % hardcoded default
     disp('WARNING: Using hardcoded NImParameters.mat')
   end
-  ImParMat
+  ImParMat;
   % if ~exist('searchRangeNum','var') || isempty(searchRangeNum)
-  if exist(CSearchRangeFile), 'file')
-    load(CSearchRangeFile);
-    CSearchRange;
+  if exist(searchRangeFile, 'file')
+    load(searchRangeFile);
+    CSearchRange; % TODO, might be an issue, figure out what this is doing
   end
       
   % yInitPos=0.30;
@@ -117,6 +118,8 @@ function NImParamRadiusGui(scansDir)
   btnNumber=7;
   yPos=0.85-(btnNumber-1)*(btnHt+spacing);
   btnPos=[xPos yPos-spacing btnWid btnHt];
+
+  % TODO, I don't think these are defined?
   try
     srchRange=ImParMat(12);
   catch  % Legacy default value was 18 before being made a user input variable (ImParMat(12)). A preferable value now might be 12 or 14.
@@ -164,7 +167,6 @@ function NImParamRadiusGui(scansDir)
     'callback',{@load_listbox});  %'uiresume(gcbf)'); 'Position', [5 100 60 20])
   
   function load_listbox(source,~)
-    global 
     userIndx=(get(source,'value'));
     userStr=(get(source,'string'));
     %scLstIndx=str2num(char(strrep(userStr(userIndx), 'Scan', '')))
@@ -239,7 +241,7 @@ function NImParamRadiusGui(scansDir)
   searchRangeNum;
 
   save(pointMapsFile, 'ImParMat');
-  save(CSearchRangeFile,'searchRangeNum');
+  save(searchRangeFile,'searchRangeNum');
 
   close
   return