1

したがって、このコードでやろうとしているのは、特定のしきい値を下回っている画像のライン上のすべてのピクセルを見つけることです。ただし、問題は、このコードが二重の for ループで実行されることです (ええ、:( )、ピクセルごとに 1 回なので、非常に遅いです。他にできることはないかと考えています。

私は MATLAB 最適化の初心者であり、基本的なことしか知らないので、いくつかのヒントが役立ちます (ループを使用しないようにするか、内部関数でスクリプトを何度も呼び出すなど)。これがうまくいかない場合は、MEX ファイルに頼らざるを得なくなる可能性があり、それは私のグループの他の研究者のために維持するのが難しくなります。ありがとうございました！

for y = 1:y_len
    for x = 1:x_len
        %//...do stuff to calc slope and offset for the line, 
                %//this can be vectorized pretty easily.

        yIndices = xIndices.*slope + offset;
        yIndices = round(yIndices);

        yIndices = yIndices + 1;
        xIndices = xIndices + 1;
        valid_points = (yIndices <= 308) & (yIndices > 0);

        %this line is bottle necking----------------------------------------
        valid_points = yIndices(valid_points)+(xIndices(valid_points)-1)*308;
        %-------------------------------------------------------------------

        valid_points = valid_points(phaseMask_R(valid_points));
        t_vals = abs(phase_R(valid_points)-currentPhase);
        point_vals = [XsR(valid_points);YsR(valid_points)] - 1;
        matchedPtsCoordsR = point_vals(:,(t_vals<phaseThreshold) |(abs(192-t_vals)<phaseThreshold));

        matchedIndex = size(matchedPtsCoordsR,2);
        if(matchedIndex ==0)
          continue
        end

        centersMinMaxR = zeros(1,matchedIndex);
        cmmIndexR = 1;
        for a = 1:matchedIndex;
          if(a==1)
            avgPosition = matchedPtsCoordsR(:,a);
            centersMinMaxR(1,1) =1;
          else
            currentPosition = matchedPtsCoordsR(:,a);


            %also very slow----------------------------------------------
            distance = sum(abs(currentPosition-avgPosition));
            %------------------------------------------------------------
            if(distance>4) % We are now likely in a different segment.
              centersMinMaxR(2,cmmIndexR) = a-1;
              cmmIndexR = cmmIndexR + 1;
              centersMinMaxR(1,cmmIndexR) = a;
            end
            avgPosition = matchedPtsCoordsR(:,a);
          end
        end

        centersMinMaxR(2,cmmIndexR) = a;
        centersR = round(sum(centersMinMaxR)/2);

        %//...do stuff with centersR
                    %//I end up concatenating all the centersR into a 
                    %//large vector arrray with the start and end of 
                    %//each segment.
4

1 に答える 1

1

まず、MatLab Profiler はあなたの親友であり、どの行がボトルネックであるかを知っているので、それについて知っていると思います。

二重ループを削除するための簡単な修正は、:コマンドを使用することです。二重ループを使用する代わりに、単一ループを使用できますが、行または列のインデックスごとにディメンション全体に沿って計算できます。簡単な例:

m = magic(2);
slope = 5;

m =
     1     3
     4     2

m(1,:) * slope  =
     5    15

m(:,1) * slope =
     5
    20

ジャグ配列を使用する代わりに、スパース配列を使用します。Matlab には組み込みのサポートがあります。

Matlab のスパース配列の作成

Matlab 疎行列操作

アップデート

スパース配列と通常配列を使用することの賛否両論について: Sparse vs Normal Array Matlab

スパース行列は、真にスパースな行列を使用する人にとって真の恩恵ですが、ほとんどの場合、25% の非ゼロは単純に「スパース」ではありません。

あなたのコードをレビューする時間がもっとあるので、さらなるアップデートを探してください :p

于 2010-07-16T20:25:41.767 に答える