Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Just use asarray #426

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

Merged
merged 1 commit into from
Mar 8, 2024
Merged

Just use asarray #426

Changes from all commits
Commits
Show all changes
1 commit
Select commit
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
18 changes: 4 additions & 14 deletions 18 fastplotlib/layouts/_gridplot.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -15,16 +15,6 @@
from ._record_mixin import RecordMixin


def to_array(a) -> np.ndarray:
if isinstance(a, np.ndarray):
return a

if not isinstance(a, list):
raise TypeError("must pass list or numpy array")

return np.array(a)


class GridPlot(Frame, RecordMixin):
def __init__(
self,
Expand Down Expand Up @@ -88,7 +78,7 @@ def __init__(
"must provide same number of subplot `names` as specified by gridplot shape"
)

self.names = to_array(names).reshape(self.shape)
self.names = np.asarray(names).reshape(self.shape)
else:
self.names = None

Expand All @@ -101,7 +91,7 @@ def __init__(
)

# list -> array if necessary
cameras = to_array(cameras).reshape(self.shape)
cameras = np.asarray(cameras).reshape(self.shape)

if cameras.shape != self.shape:
raise ValueError("Number of cameras does not match the number of subplots")
Expand Down Expand Up @@ -162,7 +152,7 @@ def __init__(

# integer ids
elif all([isinstance(item, (int, np.integer)) for item in ids_flat]):
controller_ids = to_array(controller_ids).reshape(self.shape)
controller_ids = np.asarray(controller_ids).reshape(self.shape)

else:
raise TypeError(
Expand Down Expand Up @@ -200,7 +190,7 @@ def __init__(
f"{valid_str} or instances of {[c.__name__ for c in valid_instances]}"
)

controller_types = to_array(controller_types).reshape(self.shape)
controller_types = np.asarray(controller_types).reshape(self.shape)

# make the real controllers for each subplot
self._controllers = np.empty(shape=self.shape, dtype=object)
Expand Down
Morty Proxy This is a proxified and sanitized view of the page, visit original site.