Skip to content

Commit

Permalink
feat(qc): avoid unecessary copy of CorrMag var
Browse files Browse the repository at this point in the history
This improve performance by avoiding unnecessary
copy of some possible large arrays of Correlation Magnitude fields in imosCorrMagVelocitySetQC.
  • Loading branch information
ocehugo committed Aug 25, 2020
1 parent ff061aa commit 270e53b
Showing 1 changed file with 4 additions and 8 deletions.
12 changes: 4 additions & 8 deletions AutomaticQC/imosCorrMagVelocitySetQC.m
Original file line number Diff line number Diff line change
Expand Up @@ -87,10 +87,6 @@

%Pull out correlation magnitude
sizeData = size(sample_data.variables{idCMAG{1}}.data);
cr = nan(4, sizeData(1), sizeData(2));
for j=1:4;
cr(j, :, :) = sample_data.variables{idCMAG{j}}.data;
end

% read in filter parameters
propFile = fullfile('AutomaticQC', 'imosCorrMagVelocitySetQC.txt');
Expand All @@ -109,10 +105,10 @@
flags = ones(sizeCur, 'int8')*rawFlag;

% Run QC
isub1 = (cr(1, :, :) > cmag);
isub2 = (cr(2, :, :) > cmag);
isub3 = (cr(3, :, :) > cmag);
isub4 = (cr(4, :, :) > cmag);
isub1 = sample_data.variables{idCMAG{1}}.data > cmag;
isub2 = sample_data.variables{idCMAG{2}}.data > cmag;
isub3 = sample_data.variables{idCMAG{3}}.data > cmag;
isub4 = sample_data.variables{idCMAG{4}}.data > cmag;
% test nbins bins
isub_all = isub1+isub2+isub3+isub4;
clear isub1 isub2 isub3 isub4;
Expand Down

0 comments on commit 270e53b

Please sign in to comment.