Skip to content

Commit de07207

Browse files
committed
[Matlab] Refactor Tag index use
1 parent 604ba43 commit de07207

File tree

2 files changed

+18
-14
lines changed

2 files changed

+18
-14
lines changed

+nix/Tag.m

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -54,7 +54,8 @@
5454
r = nix.Utils.open_entity(obj, 'openReferenceDataArray', id_or_name, @nix.DataArray);
5555
end
5656

57-
function r = open_reference_idx(obj, idx)
57+
function r = open_reference_idx(obj, index)
58+
idx = nix.Utils.handle_index(index);
5859
r = nix.Utils.open_entity(obj, 'openReferenceIdx', idx, @nix.DataArray);
5960
end
6061

@@ -64,7 +65,8 @@
6465
r = nix.Utils.transpose_array(data);
6566
end
6667

67-
function r = retrieve_data_idx(obj, idx)
68+
function r = retrieve_data_idx(obj, index)
69+
idx = nix.Utils.handle_index(index);
6870
fname = strcat(obj.alias, '::retrieveDataIdx');
6971
data = nix_mx(fname, obj.nix_handle, idx);
7072
r = nix.Utils.transpose_array(data);
@@ -101,7 +103,8 @@
101103
r = nix.Utils.open_entity(obj, 'openFeature', id_or_name, @nix.Feature);
102104
end
103105

104-
function r = open_feature_idx(obj, idx)
106+
function r = open_feature_idx(obj, index)
107+
idx = nix.Utils.handle_index(index);
105108
r = nix.Utils.open_entity(obj, 'openFeatureIdx', idx, @nix.Feature);
106109
end
107110

@@ -111,7 +114,8 @@
111114
r = nix.Utils.transpose_array(data);
112115
end
113116

114-
function r = retrieve_feature_data_idx(obj, idx)
117+
function r = retrieve_feature_data_idx(obj, index)
118+
idx = nix.Utils.handle_index(index);
115119
fname = strcat(obj.alias, '::featureRetrieveDataIdx');
116120
data = nix_mx(fname, obj.nix_handle, idx);
117121
r = nix.Utils.transpose_array(data);

tests/TestTag.m

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -440,9 +440,9 @@
440440
t.add_feature(d2, nix.LinkType.Untagged);
441441
t.add_feature(d3, nix.LinkType.Indexed);
442442

443-
assert(f.blocks{1}.tags{1}.open_feature_idx(0).linkType == nix.LinkType.Tagged);
444-
assert(f.blocks{1}.tags{1}.open_feature_idx(1).linkType == nix.LinkType.Untagged);
445-
assert(f.blocks{1}.tags{1}.open_feature_idx(2).linkType == nix.LinkType.Indexed);
443+
assert(f.blocks{1}.tags{1}.open_feature_idx(1).linkType == nix.LinkType.Tagged);
444+
assert(f.blocks{1}.tags{1}.open_feature_idx(2).linkType == nix.LinkType.Untagged);
445+
assert(f.blocks{1}.tags{1}.open_feature_idx(3).linkType == nix.LinkType.Indexed);
446446
end
447447

448448
%% Test: Open reference by ID or name
@@ -477,9 +477,9 @@
477477
t.add_reference(d2);
478478
t.add_reference(d3);
479479

480-
assert(strcmp(f.blocks{1}.tags{1}.open_reference_idx(0).name, d1.name));
481-
assert(strcmp(f.blocks{1}.tags{1}.open_reference_idx(1).name, d2.name));
482-
assert(strcmp(f.blocks{1}.tags{1}.open_reference_idx(2).name, d3.name));
480+
assert(strcmp(f.blocks{1}.tags{1}.open_reference_idx(1).name, d1.name));
481+
assert(strcmp(f.blocks{1}.tags{1}.open_reference_idx(2).name, d2.name));
482+
assert(strcmp(f.blocks{1}.tags{1}.open_reference_idx(3).name, d3.name));
483483
end
484484

485485
%% Test: Set metadata
@@ -582,7 +582,7 @@
582582
assert(~isempty(strfind(ME.message, 'out of bounds')), 'Invalid index failed');
583583
end
584584

585-
retData = t.retrieve_data_idx(0);
585+
retData = t.retrieve_data_idx(1);
586586
assert(size(retData, 2) == t.extent, 'Extent check failed');
587587
assert(retData(1) == raw(t.position + 1), 'Position check failed');
588588
end
@@ -650,22 +650,22 @@
650650
df = b.create_data_array_from_data('testUntagged', 'nixDataArray', rawFeature);
651651
df.append_sampled_dimension(1);
652652
t.add_feature(df, nix.LinkType.Untagged);
653-
retData = t.retrieve_feature_data_idx(0);
653+
retData = t.retrieve_feature_data_idx(1);
654654
assert(size(retData, 2) == size(rawFeature, 2), 'Untagged size check fail');
655655

656656
% test retrieve tagged feature data
657657
df = b.create_data_array_from_data('testTagged', 'nixDataArray', rawFeature);
658658
df.append_sampled_dimension(1);
659659
t.add_feature(df, nix.LinkType.Tagged);
660-
retData = t.retrieve_feature_data_idx(1);
660+
retData = t.retrieve_feature_data_idx(2);
661661
assert(size(retData, 2) == t.extent, 'Tagged Extent check fail');
662662
assert(retData(1) == rawFeature(t.position + 1), 'Tagged Position check fail');
663663

664664
% test retrieve indexed feature data
665665
df = b.create_data_array_from_data('testIndexed', 'nixDataArray', rawFeature);
666666
df.append_sampled_dimension(1);
667667
t.add_feature(df, nix.LinkType.Indexed);
668-
retData = t.retrieve_feature_data_idx(2);
668+
retData = t.retrieve_feature_data_idx(3);
669669
assert(size(retData, 2) == size(rawFeature, 2), 'Indexed size check fail');
670670

671671
try

0 commit comments

Comments
 (0)