Skip to content

Commit

Permalink
notes
Browse files Browse the repository at this point in the history
  • Loading branch information
mavaylon1 committed Feb 19, 2025
1 parent 7049883 commit 7bbeba4
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions src/hdmf/common/table.py
Original file line number Diff line number Diff line change
Expand Up @@ -203,7 +203,10 @@ def get(self, arg, **kwargs):
arg = np.where(arg)[0]
indices = arg
ret = list()
if len(indices) > 0: # This is for test_to_hierarchical_dataframe_empty_tables
if len(indices) > 0:
# Note: len(indices) == 0 for test_to_hierarchical_dataframe_empty_tables.
# This is an edge case test for to_hierarchical_dataframe() on empty tables.
# When len(indices) == 0, ret is expected to be an empty list, defiend above.

Check failure on line 209 in src/hdmf/common/table.py

View workflow job for this annotation

GitHub Actions / Check for spelling errors

defiend ==> defined
try:
data = self.target.get(slice(None), **kwargs)
slices = [self.__get_slice(i) for i in indices]
Expand All @@ -213,7 +216,8 @@ def get(self, arg, **kwargs):
ret = [data[s] for s in slices]
except IndexError:
"""
Note: TODO: test_to_hierarchical_dataframe_indexed_dtr_on_last_level
Note: TODO: test_to_hierarchical_dataframe_indexed_dtr_on_last_level.
This is the old way to get the data and not an untested feature.
"""
for i in indices:
ret.append(self.__getitem_helper(i, **kwargs))
Expand Down

0 comments on commit 7bbeba4

Please sign in to comment.