How to make this simple script faster?

1 view (last 30 days)
Each of the Sta{i}.test is a one-column data with unknown number of rows. As you know, to keep expanding Variable X is not the most efficient way of doing this.
Is there a way I can get my X values faster without writing a loop?
X = [];
for i = a:b
if isfield(Sta{i}, 'test')
X = [X; Sta{i}.test];
end
end
Many thanks!
  1 Comment
Image Analyst
Image Analyst on 3 Feb 2020
Edited: Image Analyst on 3 Feb 2020
Are you saying that sometimes the structure inside the cell might not have a field called "test"? What if you just allocated X as, say, 10 million elements or way more than you ever expect to have, then crop to the proper length after the loop. See my Answer below (scroll down).

Sign in to comment.

Accepted Answer

Image Analyst
Image Analyst on 3 Feb 2020
Edited: Image Analyst on 3 Feb 2020
Does this work:
X = zeros(10000000, 1);
a = 1;
b = length(Sta);
lastIndex = 0
for i = a:b
if isfield(Sta{i}, 'test')
t = Sta{i}.test;
lt = length(t);
X(lastIndex+1:lastIndex+lt) = t;
lastIndex = lastIndex + lt;
end
end
X = X(1:lastIndex);
What is the size of Sta? How many elements does it have?
  3 Comments
Image Analyst
Image Analyst on 3 Feb 2020
Edited: Image Analyst on 3 Feb 2020
It might be. I experimented with doing things like (:) and {:} and putting stuff in brackets and using vertcat() but I just couldn't find the exact syntax that would do it. You'd have to experiment around some.
Can you store them in a more convenient way in the first place?
Leon
Leon on 3 Feb 2020
Any recommendation on what's the best way to store them in order to access them in the most convenient way?
In a matrix named A like this:
Sta#, Var1, Var2
1 35 127
1 21 256
1 11 222
2 15 188
2 12 236
3 27 333
4 35 467
5 11 222
5 23 376
Then how do I quickly get all the rows with a Sta# between 2 and 4, for example?
Ind = A(:,1)>=2 & A(:,1)<=4;
B = A(Ind, :);
Is there a better setup? Many thanks!

Sign in to comment.

More Answers (1)

David Hill
David Hill on 3 Feb 2020
You could preallocate X with nan to the largest expected and then delete the excess nan's at the end.
X = nan(10000000,1);
count=1;
for i = a:b
if isfield(Sta{i}, 'test')
temp=count+length(Sta{i}.test);
X(count:temp-1) = Sta{i}.test;
count=temp;
end
end
X=X(~isnan(X));
  1 Comment
Leon
Leon on 3 Feb 2020
Many thanks for the solution, David! So bad I can only accept one answer.

Sign in to comment.

Tags

Products


Release

R2019b

Community Treasure Hunt

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

Start Hunting!