Skip to content
Snippets Groups Projects
Commit d398f5e1 authored by Maximilian Schnabel's avatar Maximilian Schnabel
Browse files

Revert "added warning to setProperties, if invalid input parameters are given"

This reverts commit 1c8f3a2e
parent 1358c13c
Branches
No related tags found
2 merge requests!7Keep going if mountpoint exists (persistent),!6Docs NXT compatibility
...@@ -347,15 +347,17 @@ classdef Motor < MaskedHandle & dynamicprops ...@@ -347,15 +347,17 @@ classdef Motor < MaskedHandle & dynamicprops
% * This is a pretty 'heavy' function, as it tests if both motors are % * This is a pretty 'heavy' function, as it tests if both motors are
% connected AND aren't running, wasting four packets, keep that in mind. % connected AND aren't running, wasting four packets, keep that in mind.
% %
% Example: %::
% b = EV3(); % |br| %
% b.connect('usb'); % |br| % Example:
% m = b.motorA; % |br| % brick = EV3();
% slave = b.motorB; % |br| % brick.connect('usb');
% m.power = 50; % |br| % motor = brick.motorA;
% m.syncedStart(slave); % |br| % slave = brick.motorB;
% % Do stuff |br| % motor.power = 50;
% m.stop(); % |br| % motor.syncedStart(slave);
% % Do stuff
% motor.stop();
% %
% See also MOTOR.STOP, MOTOR.SYNCEDSTOP / :meth:`stop`, :meth:`syncedStop` % See also MOTOR.STOP, MOTOR.SYNCEDSTOP / :meth:`stop`, :meth:`syncedStop`
...@@ -713,20 +715,21 @@ classdef Motor < MaskedHandle & dynamicprops ...@@ -713,20 +715,21 @@ classdef Motor < MaskedHandle & dynamicprops
% power (numeric in [-100,100]): *[OPTIONAL]* % power (numeric in [-100,100]): *[OPTIONAL]*
% batteryMode ('Voltage'|'Percentage'): *[OPTIONAL]* % batteryMode ('Voltage'|'Percentage'): *[OPTIONAL]*
% %
% Example: % ::
% b = EV3(); % |br| %
% b.connect('bt', 'serPort', '/dev/rfcomm0'); % |br| % Example:
% b.motorA.setProperties('debug', 'on', 'power', 50, 'limitValue', 720, 'speedRegulation', 'on'); % |br| % brick = EV3();
% % Instead of: b.motorA.debug = 'on'; |br| % brick.connect('bt', 'serPort', '/dev/rfcomm0');
% % b.motorA.power = 50; |br| % brick.motorA.setProperties('debug', 'on', 'power', 50, 'limitValue', 720, 'speedRegulation', 'on');
% % b.motorA.limitValue = 720; |br| % % Instead of: brick.motorA.debug = 'on';
% % b.motorA.speedRegulation = 'on'; |br| % % brick.motorA.power = 50;
% % brick.motorA.limitValue = 720;
% % brick.motorA.speedRegulation = 'on';
% %
p = inputParser(); p = inputParser();
p.KeepUnmatched = 1; p.KeepUnmatched = 1;
% Set default values % Set default values
if motor.init if motor.init
defaultDebug = 0; defaultDebug = 0;
...@@ -764,17 +767,7 @@ classdef Motor < MaskedHandle & dynamicprops ...@@ -764,17 +767,7 @@ classdef Motor < MaskedHandle & dynamicprops
% Parse... % Parse...
p.parse(varargin{:}); p.parse(varargin{:});
if ~isempty(fieldnames(p.Unmatched))
A = fieldnames(p.Unmatched);
warn = 'The following input parameters were invalid: ';
warn = [warn A{1}];
for i = 2:length(A)
warn = [warn ', ' A{i}];
end
warning(warn)
end
% Set properties % Set properties
if motor.init if motor.init
motor.port = p.Results.port; motor.port = p.Results.port;
...@@ -1049,9 +1042,10 @@ classdef Motor < MaskedHandle & dynamicprops ...@@ -1049,9 +1042,10 @@ classdef Motor < MaskedHandle & dynamicprops
end end
% If motor is *busily* running, stop it. That avoids suicidal stuff like: % If motor is *busily* running, stop it. That avoids suicidal stuff like:
% b = EV3(); b.connect('usb'); % brick = EV3();
% b.motorA.start(); % brick.connect('usb');
% b.disconnect(); -> Motor still running and cannot directly be stopped anymore % brick.motorA.start();
% brick.disconnect(); -> Motor still running and cannot directly be stopped anymore
if motor.isRunning if motor.isRunning
motor.stop(); motor.stop();
end end
...@@ -1073,4 +1067,4 @@ classdef Motor < MaskedHandle & dynamicprops ...@@ -1073,4 +1067,4 @@ classdef Motor < MaskedHandle & dynamicprops
end end
end end
end end
end end
\ No newline at end of file
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment