Skip to content

Commit 94d4f77

Browse files
committed
ruff linting + fix typing issues
1 parent d51dc94 commit 94d4f77

File tree

1 file changed

+21
-30
lines changed

1 file changed

+21
-30
lines changed

src/scportrait/pipeline/featurization.py

Lines changed: 21 additions & 30 deletions
Original file line numberDiff line numberDiff line change
@@ -119,7 +119,9 @@ def _get_nmasks(self):
119119
for file in self.extraction_file:
120120
with h5py.File(file, "r") as f:
121121
n_masks.append(f["n_masks"][()].item())
122-
assert (x == n_masks[0] for x in n_masks), "number of masks are not consistent over all passed HDF5 files."
122+
assert (
123+
x == n_masks[0] for x in n_masks
124+
), "number of masks are not consistent over all passed HDF5 files."
123125
self.n_masks = n_masks[0]
124126
try:
125127
self.n_masks = h5py.File(self.extraction_file, "r")["n_masks"][()].item()
@@ -188,7 +190,7 @@ def _setup_inference_device(self):
188190
self.inference_device = self._detect_automatic_inference_device()
189191
self.log(f"Automatically configured inferece device to {self.inference_device}")
190192

191-
def _general_setup(self, extraction_dir: str, return_results: bool = False):
193+
def _general_setup(self, extraction_dir: str | list[str], return_results: bool = False):
192194
"""Helper function to execute all setup functions that are common to all featurization steps."""
193195

194196
self.extraction_file = extraction_dir
@@ -834,11 +836,7 @@ def _setup(self, extraction_dir: str, return_results: bool):
834836
self._setup_encoders()
835837
self._setup_transforms()
836838

837-
def process(self,
838-
extraction_dir: str,
839-
labels: int | list[int] = 0,
840-
size: int = 0,
841-
return_results: bool = False):
839+
def process(self, extraction_dir: str, labels: int | list[int] = 0, size: int = 0, return_results: bool = False):
842840
"""
843841
Perform classification on the provided HDF5 dataset.
844842
@@ -919,7 +917,7 @@ class based on the previous single-cell extraction. Therefore, only the second a
919917

920918
self.dataloader = self.generate_dataloader(
921919
extraction_dir,
922-
labels = labels,
920+
labels=labels,
923921
selected_transforms=self.transforms,
924922
size=size,
925923
dataset_class=self.DEFAULT_DATA_LOADER,
@@ -1012,11 +1010,7 @@ def _setup(self, extraction_dir: str, return_results: bool):
10121010

10131011
self._load_models()
10141012

1015-
def process(self,
1016-
extraction_dir: str,
1017-
labels: int | list[int] = 0,
1018-
size: int = 0,
1019-
return_results: bool = False):
1013+
def process(self, extraction_dir: str, labels: int | list[int] = 0, size: int = 0, return_results: bool = False):
10201014
"""
10211015
Function called to perform classification on the provided HDF5 dataset.
10221016
@@ -1075,7 +1069,7 @@ class based on the previous single-cell extraction. Therefore, no parameters nee
10751069

10761070
self.dataloader = self.generate_dataloader(
10771071
extraction_dir,
1078-
labels = labels,
1072+
labels=labels,
10791073
selected_transforms=self.transforms,
10801074
size=size,
10811075
dataset_class=self.DEFAULT_DATA_LOADER,
@@ -1149,7 +1143,9 @@ def _get_channel_specs(self):
11491143
for file in self.extraction_file:
11501144
with h5py.File(file, "r") as f:
11511145
channel_names.append(list(f["channel_information"][:].astype(str)))
1152-
assert (x == channel_names[0] for x in channel_names), "Channel names are not consistent over all passed HDF5 files."
1146+
assert (
1147+
x == channel_names[0] for x in channel_names
1148+
), "Channel names are not consistent over all passed HDF5 files."
11531149
self.channel_names = channel_names[0]
11541150
else:
11551151
if "channel_names" in self.project.__dict__.keys():
@@ -1367,16 +1363,14 @@ def __init__(self, *args, **kwargs):
13671363

13681364
self.channel_selection = None # ensure that all images are passed to the function
13691365

1370-
def _setup(self, extraction_dir: str, return_results: bool):
1366+
def _setup(self, extraction_dir: str | list[str], return_results: bool):
13711367
self._general_setup(extraction_dir=extraction_dir, return_results=return_results)
13721368
self._setup_transforms()
13731369
self._get_channel_specs()
13741370

1375-
def process(self,
1376-
extraction_dir: str | list[str],
1377-
labels: int | list[int] = 0,
1378-
size: int = 0,
1379-
return_results: bool = False):
1371+
def process(
1372+
self, extraction_dir: str | list[str], labels: int | list[int] = 0, size: int = 0, return_results: bool = False
1373+
):
13801374
"""
13811375
Perform featurization on the provided HDF5 dataset.
13821376
@@ -1435,7 +1429,7 @@ def process(self,
14351429

14361430
self.dataloader = self.generate_dataloader(
14371431
extraction_dir,
1438-
labels = labels,
1432+
labels=labels,
14391433
selected_transforms=self.transforms,
14401434
size=size,
14411435
dataset_class=self.DEFAULT_DATA_LOADER,
@@ -1490,26 +1484,23 @@ def _setup_channel_selection(self):
14901484
self.channel_selection = [0, self.channel_selection]
14911485
return
14921486

1493-
def _setup(self, extraction_dir: str, return_results: bool):
1487+
def _setup(self, extraction_dir: str | list[str], return_results: bool):
14941488
self._general_setup(extraction_dir=extraction_dir, return_results=return_results)
14951489
self._setup_channel_selection()
14961490
self._setup_transforms()
14971491
self._get_channel_specs()
14981492

1499-
def process(self,
1500-
extraction_dir: str | list[str],
1501-
labels: int | list[int] = 0,
1502-
size=0,
1503-
return_results: bool = False):
1504-
1493+
def process(
1494+
self, extraction_dir: str | list[str], labels: int | list[int] = 0, size=0, return_results: bool = False
1495+
):
15051496
self.log(f"Started CellFeaturization of selected channel {self.channel_selection}.")
15061497

15071498
# perform setup
15081499
self._setup(extraction_dir=extraction_dir, return_results=return_results)
15091500

15101501
self.dataloader = self.generate_dataloader(
15111502
extraction_dir,
1512-
labels = labels,
1503+
labels=labels,
15131504
selected_transforms=self.transforms,
15141505
size=size,
15151506
dataset_class=self.DEFAULT_DATA_LOADER,

0 commit comments

Comments
 (0)