Azzera filtri
Azzera filtri

Making a code runs faster

2 visualizzazioni (ultimi 30 giorni)
Ronaldo
Ronaldo il 2 Set 2013
How can I make the following code runs faster (This a part of a larger code). I do not want to use parfor.
for rotation=1:360
RHO(:,1)=(CL*Landa)./T(tnum).LoAn(:,1);
THETA=T(tnum).LoAn(:,3);
X(:,1)=RHO.*cos(deg2rad(THETA(:,1)+rotation));
Y(:,1)=RHO.*sin(deg2rad(THETA(:,1)+rotation));
XP=round(72+X);
YP=round(72+Y);
mat=zeros(144,144);
for i=1:size(XP,1)
a=XP(i,1);
b=YP(i,1);
if a>0 & a<145 & b>0 & b<145
mat(a,b)= T(tnum).LoAn(i,2);
end
end
end
  2 Commenti
Walter Roberson
Walter Roberson il 2 Set 2013
Isn't your RHO the same for each iteration? If so then compute it outside the loop. Same for THETA.
To confirm: LoAn is a 2D array, and not just a row vector ? so size(LoAn,1) > 1 ?
Roger Stafford
Roger Stafford il 2 Set 2013
It looks as though your purpose in this code is to set up the array 'mat'. However, on each trip through the outer for-loop this array is completely erased and refilled with the information from the next 'rotation' angle. What is the sense in that? Only the last 360 degree angle would have any effect on the final values in 'mat'. Is it possible you are doing something with 'mat' which you haven't shown here before returning for another rotation angle? We need to know such things if we are to help you effectively.

Accedi per commentare.

Risposte (1)

Azzi Abdelmalek
Azzi Abdelmalek il 2 Set 2013
Modificato: Azzi Abdelmalek il 2 Set 2013
  1 Commento
Ronaldo
Ronaldo il 2 Set 2013
In other part of the code, I did pre-allocating. I like to make more parallel calculations. Since I am going to use spmd, I try to avoid parfor loops.

Accedi per commentare.

Categorie

Scopri di più su Loops and Conditional Statements in Help Center e File Exchange

Community Treasure Hunt

Find the treasures in MATLAB Central and discover how the community can help you!

Start Hunting!

Translated by