Better documentation; update arg name to update_frequency; removed redundant obj input parsing

Author: austin2118ace

Functions/Modules/Analog Input/BpodAnalogIn.m

@@ -893,24 +893,24 @@ function scope(obj)
         end
 
         function scope_StartStop(obj, varargin)
-            % scope_StartStop() toggles data acquisition by the scope()
-            % GUI. It is called by a start button on the GUI, but it can also be 
-            % called from a user protocol file to start analog data logging with
-            % online monitoring.
+        % scope_StartStop   This function toggles data acquisition when using the oscilloscope functionality of the AnalogIn module. 
+        % The function is called by the start button on the oscilloscope GUI, but can also be called from a
+        % user protocol to start recording data with online monitoring via the GUI. 
+
+        % :param update_frequency: Update frequency, in seconds (s), for the oscilloscope GUI during data acquisition, defaults to 0.05 (50 ms)
+        % :type update_frequency: float, optional
+
 
             % Default GUI update time is 0.05s
             default_update_time = 0.05;
-            oscope_time = 0;
+            update_frequency = 0;
 
             ip = inputParser;
             valid_update_time = @(time) isfloat(time) && isscalar(time) && (time > 0);  % Update time must be a float, scalar, and greater than zero
-            valid_obj = @(x) isa(x, 'BpodAnalogIn'); % Have to also check obj is an instance of BpodAnalogIn
-
-            addRequired(ip, 'obj', valid_obj);  % instance is a required argument
-            addOptional(ip, 'oscope_time', default_update_time, valid_update_time); % oscope_time is optional, if the passed value is invalid, the default is used
+            addOptional(ip, 'update_frequency', default_update_time, valid_update_time); % update_frequency is optional, if the passed value is invalid, the default is used
 
-            parse(ip, obj, varargin{:});  % parse the args
-            oscope_time = ip.Results.oscope_time; % assuming all is well, grab the value from the parser
+            parse(ip, varargin{:});  % parse the args
+            update_frequency = ip.Results.update_frequency; % assuming all is well, grab the value from the parser
 
             scopeReady = 1;
             if ~isfield(obj.UIhandles, 'OscopeFig')
@@ -942,7 +942,7 @@ function scope_StartStop(obj, varargin)
                     end
                     obj.UIdata.SweepPos = 1;
                     obj.startUSBStream;
-                    obj.Timer = timer('TimerFcn',@(h,e)obj.updatePlot(), 'ExecutionMode', 'fixedRate', 'Period', oscope_time);
+                    obj.Timer = timer('TimerFcn',@(h,e)obj.updatePlot(), 'ExecutionMode', 'fixedRate', 'Period', update_frequency);
                     start(obj.Timer);
                 else
                     stop(obj.Timer);