Skip to content

Commit 2f58a30

Browse files
committed
hotfix: test lists and numpy arrays
1 parent 4ed44f2 commit 2f58a30

8 files changed

Lines changed: 11 additions & 6 deletions

File tree

-10.9 KB
Loading
-5.76 KB
Loading
-2.15 KB
Loading
215 Bytes
Loading
8.56 KB
Loading
318 Bytes
Loading
-367 Bytes
Loading

optical_toolkit/visualize/visualize_images.py

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -83,12 +83,17 @@ def summarize_images(
8383
Returns:
8484
plt.Figure: The matplotlib figure object for the summary grid.
8585
"""
86-
if not images or not targets:
87-
raise ValueError("Images and targets must be non-empty.")
88-
if len(images) != len(targets):
89-
raise ValueError("Length of images and targets must match.")
90-
if num_images_per_class <= 0:
91-
raise ValueError("num_images_per_class must be positive.")
86+
if isinstance(images, np.ndarray):
87+
if images.size == 0:
88+
raise ValueError("Images array cannot be empty.")
89+
elif len(images) == 0:
90+
raise ValueError("Images list cannot be empty.")
91+
92+
if isinstance(targets, np.ndarray):
93+
if targets.size == 0:
94+
raise ValueError("Targets array cannot be empty.")
95+
elif len(targets) == 0:
96+
raise ValueError("Targets list cannot be empty.")
9297

9398
class_images: Dict[int, List[np.ndarray]] = defaultdict(list)
9499
for img, label in zip(images, targets):

0 commit comments

Comments
 (0)