Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix getRow method for dynamic table #648

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Fix getRow method for dynamic table #648

Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
6096243
Permute nd arrays to get correct shape for table
ehennestad Jul 6, 2024
283f79a
Update getRow.m
ehennestad Jul 6, 2024
255ddb2
Merge branch 'master' into 576-fix-get-row-for-arrays
ehennestad Sep 12, 2024
62430eb
Add test for previous failure point
ehennestad Sep 12, 2024
fc46950
Merge branch 'master' into 576-fix-get-row-for-arrays
ehennestad Sep 12, 2024
bed7e73
Merge branch 'master' into 576-fix-get-row-for-arrays
ehennestad Dec 12, 2024
dff5ca6
Update getRow.m
ehennestad Dec 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions +types/+util/+dynamictable/getRow.m
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -46,11 +46,12 @@
row{i} = select(DynamicTable, indexNames, ind);

if ~istable(row{i})
% transpose row vectors
if isrow(row{i})
row{i} = row{i} .';
% or permute arrays to place last dimension first
elseif ~ismatrix(row{i}) % i.e nd array where n >= 3
if iscolumn(row{i})
% keep column vectors as is
elseif isrow(row{i})
row{i} = row{i} .'; % transpose row vectors
elseif ndims(row{i}) >= 2 % i.e nd array where ndims >= 2
% permute arrays to place last dimension first
array_size = size(row{i});
num_rows = numel(ind);

Expand Down
Loading