MATLAB: Using a script to change uicontrol values in a gui

callbackguideMATLABmatlab functionmatlab gui

I have a code that initializes a GUI, and then take serial data, compares it to excel master files, and then changes the colors of some uipanels in the gui. this is the code, the comments are generally prototyping or holdovers from other codes I incorporated:

 delete(instrfind('Port', 'COM3')); 
tag = serial('COM3'); %check which port is used
fopen(tag);
MyGUI;
BOX = char(zeros(2,14)); 
i=1;
c=0;
TrueValueData = 'C:\MasterCodes.xlsx';
[~,~,TrueValMat] = xlsread(TrueValueData); % Creates matrix filled with the correct values, 
                                           % indexed by box, which is the first row
                                           % all proceeding rows are the master value
function result(handles)
    for i=1:9223372036854775807 %just keeps looping, will probably replace with a push button to kill
     if i>10 %first couple reads are filled with unicode nonsense, this skips that stage
    
     readData = fscanf(tag);
    if length(readData)>12
        
       BOX(str2num(readData(8)),1:14)= readData(11:24); % these numbers just give us what we want; 
                                                   % tags come in initially with some gobbledy-gook     
    end
%     
%      if(length(readData)>10) %if chip has been read
%          
%      ReadChips
  
          if strcmp(TrueValMat{2,1}, BOX(1,:))
             set(handles.uipanel1, 'BackgroundColor', 'green');
          else 
             set(handles.uipanel1, 'BackgroundColor', 'red');
          end
        
            if strcmp(TrueValMat{2,2}, BOX(2,:))
             set(handles.uipanel2, 'Backgroundcolor', 'green');
            else 
             set(handles.uipanel2, 'Backgroundcolor', 'red');
            end
            if strcmp(TrueValMat{2,1}, BOX(1,:))...
                    && strcmp(TrueValMat{2,2}, BOX(2,:)) == 1
                break
            end
     end
    end
    end
function uipanel1_Callback(hObject, eventdata, handles)
result;
function uipanel2_Callback(hObject, eventdata, handles)
result;
end
end

The problem is that this doesnt actually do anything, the GUI comes up, but the loop doesn't iterate, and the colors don't change in the GUI.

edit: added in loop break point

Best Answer

Just build all that code into the m-file that GUIDE made for you. Why control your GUI from a separate, external script? If you have some settings that you need to specify, just use a pulldown menu to bring up a separate sub-GUI then return the settings to the main GUI.

Related Solutions

MATLAB: Feval error in GUI implementation

You have

function uipanel1_CreateFcn(hObject, eventdata, handles1)
 function Reader1Correct
     set(handles1.uipanel1, 'BackgroundColor', 'g');
    end
    function Reader1Incorrect
        set(handles1.uipanel1, 'BackgroundColor', 'r');
    end
end

That defines nested functions inside upanel1_CreateFcn (but does not call those functions.) It is not possible to define nested functions inside code that uses "function" without the matching "end" statement, so you would have had to add "end" statements to a number of places in the code.

But when you did that, you did not pay attention to the indentation, and you mis-nested. The "end" statement to terminate WorkingGUI2_OpeningFcn is not until much later, so several functions ended up nesting inside WorkingGUI2_OpeningFcn -- including WorkingGUI2_Outputfcn, so the reference to it in the OpenFcn cannot find it because it is not visible from the scope of the OpenFcn

Nested functions do not play well with GUIDE. Once you insert any nested function, then even if you manage to put all the "end" in at the correct places yourself, GUIDE can no longer manage the modified .m file.

If you need nested functions, then you should probably put the enclosing function into a different .m file, and the defining function should not be the direct target of a callback, and should instead be called from a callback.

MATLAB: Iterate through multiple serial ports, one after another (not simultaneously)

portlist = {'COM3', 'COM4', 'COM7', 'COM12'};
nport = length(portlist);
tags = cell{1, nport};
cleanups = cell{1, nport};
for portidx = 1 : nport
    delete(instrfind('Port', portlist{portidx})); % removes possibility for 'Port not available' error
    tags{portidx} = serial(portlist{portidx}); %initializes the port to be used
    fopen(tag); %opens th eport
    cleanups{portidx} = onCleanup(@() fclose(portlist{portidx}));
end
BOX = char(zeros(2,14)); % matrix to be populated with incoming serial data
TrueValueData = 'C:\RfidChipTrueValues.xlsx'; 
% location of stored master tags
[~,~,TrueValMat] = xlsread(TrueValueData); % reads our excel file into the proper format
% Creates matrix filled with the correct values
% indexed by box, which is the first row
% all proceeding rows are the master value
for i=1:inf
    for portidx = 1 : nport
        nbase = portidx * 2 - 1;
        for n = nbase:nbase+1
            if i>10 % positive reading
                readData = fscanf(tags{portidx});;
                if length(readData)>12
                    BOX(str2double(readData(8)),1:14)= readData(11:24); 
                    if strcmp(TrueValMat{2,n}, BOX(n,:)) %cannot sub-index to CELL types normally, must use this method
                        set(handles.uipanels(n), 'BackgroundColor', 'g');
                    else
                        set(handles.uipanels(n), 'BackgroundColor', 'r');
                    end
                    drawnow % allows GUI to update the appearance whenever the loop completes.
                    if strcmp(TrueValMat{2,1}, BOX(1,:))...
                            && strcmp(TrueValMat{2,2}, BOX(2,:)) == 1
                        set(handles.pushbutton5, 'enable', 'on');
                    end
                end
            end
        end
    end
end

If you want something different to be done for the different ports, then you can test the portidx value.

Related Question