Skip to content

Commit 07273dd

Browse files
[pre-commit.ci] auto fixes from pre-commit.com hooks
for more information, see https://pre-commit.ci
1 parent b558927 commit 07273dd

File tree

3 files changed

+15
-42
lines changed

3 files changed

+15
-42
lines changed

.isort.cfg

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,2 +1,2 @@
11
[settings]
2-
known_third_party = dask,numpy,ome_zarr,omero,omero_rois,omero_zarr,pytest,setuptools,skimage,zarr
2+
known_third_party = dask,numpy,ome_zarr,omero,omero_rois,omero_sys_ParametersI,omero_zarr,pytest,setuptools,skimage,zarr

src/omero_zarr/cli.py

Lines changed: 9 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
from zarr.hierarchy import open_group
3030
from zarr.storage import FSStore
3131

32+
from .extinfo import external_info_str, get_extinfo, get_images, set_external_info
3233
from .masks import (
3334
MASK_DTYPE_SIZE,
3435
MaskSaver,
@@ -41,12 +42,6 @@
4142
image_to_zarr,
4243
plate_to_zarr,
4344
)
44-
from .extinfo import (
45-
get_images,
46-
set_external_info,
47-
get_extinfo,
48-
external_info_str
49-
)
5045

5146
HELP = """Export data in zarr format.
5247
@@ -205,8 +200,7 @@ def _configure(self, parser: Parser) -> None:
205200
polygons.add_argument(
206201
"--label-name",
207202
help=(
208-
"Name of the array that will be stored. Ignored for "
209-
"--style=split"
203+
"Name of the array that will be stored. Ignored for " "--style=split"
210204
),
211205
default="0",
212206
)
@@ -236,8 +230,7 @@ def _configure(self, parser: Parser) -> None:
236230
masks.add_argument(
237231
"--label-name",
238232
help=(
239-
"Name of the array that will be stored. Ignored for "
240-
"--style=split"
233+
"Name of the array that will be stored. Ignored for " "--style=split"
241234
),
242235
default="0",
243236
)
@@ -302,10 +295,7 @@ def _configure(self, parser: Parser) -> None:
302295
export.add_argument(
303296
"--max_workers",
304297
default=None,
305-
help=(
306-
"Maximum number of workers (only for "
307-
"use with bioformats2raw)"
308-
),
298+
help=("Maximum number of workers (only for " "use with bioformats2raw)"),
309299
)
310300
export.add_argument(
311301
"object",
@@ -428,8 +418,7 @@ def extinfo(self, args: argparse.Namespace) -> None:
428418
args.entityType,
429419
int(args.entityId),
430420
)
431-
img = self.gateway.getUpdateService()\
432-
.saveAndReturnObject(img)
421+
img = self.gateway.getUpdateService().saveAndReturnObject(img)
433422
self.ctx.out(
434423
f"Set ExternalInfo for image ({img.id._val}) "
435424
f"{img.name._val}:\n"
@@ -443,16 +432,14 @@ def extinfo(self, args: argparse.Namespace) -> None:
443432
elif args.reset:
444433
if extinfo:
445434
img.details.externalInfo = None
446-
img = self.gateway.getUpdateService()\
447-
.saveAndReturnObject(img)
435+
img = self.gateway.getUpdateService().saveAndReturnObject(img)
448436
self.ctx.out(
449437
f"Removed ExternalInfo from image "
450438
f"({img.id._val}) {img.name._val}"
451439
)
452440
else:
453441
self.ctx.out(
454-
f"Image ({img.id._val}) {img.name._val} "
455-
"has no ExternalInfo"
442+
f"Image ({img.id._val}) {img.name._val} " "has no ExternalInfo"
456443
)
457444
else:
458445
if extinfo:
@@ -463,8 +450,7 @@ def extinfo(self, args: argparse.Namespace) -> None:
463450
)
464451
else:
465452
self.ctx.out(
466-
f"Image ({img.id._val}) {img.name._val} has no "
467-
"ExternalInfo"
453+
f"Image ({img.id._val}) {img.name._val} has no " "ExternalInfo"
468454
)
469455

470456
def _lookup(
@@ -477,9 +463,7 @@ def _lookup(
477463
self.ctx.die(110, f"No such {otype}: {oid}")
478464
return obj
479465

480-
def _bf_export(
481-
self, image: BlitzObjectWrapper, args: argparse.Namespace
482-
) -> None:
466+
def _bf_export(self, image: BlitzObjectWrapper, args: argparse.Namespace) -> None:
483467
if args.bfpath:
484468
abs_path = Path(args.bfpath)
485469
elif image.getInplaceImport():

src/omero_zarr/extinfo.py

Lines changed: 5 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -1,25 +1,17 @@
11
import re
22

33
from omero.gateway import BlitzGateway, BlitzObjectWrapper, ImageWrapper
4-
from omero.model import (
5-
Dataset,
6-
ExternalInfoI,
7-
Image,
8-
ImageI,
9-
Plate,
10-
Project,
11-
Screen,
12-
)
4+
from omero.model import Dataset, ExternalInfoI, Image, ImageI, Plate, Project, Screen
135
from omero.rtypes import rlong, rstring
146
from omero_sys_ParametersI import ParametersI
157

16-
178
# Regex to match well positions (eg. A1)
189
WELL_POS_RE = re.compile(r"(?P<row>\D+)(?P<col>\d+)")
1910
# Regex to match the metadata.xml (could be any xml under xyz.zarr/ directory,
2011
# not only xyz.zarr/OME/METADATA.ome.xml)
2112
METADATA_XML_RE = re.compile(r".+\.zarr\/(.+\.xml)")
2213

14+
2315
def get_extinfo(conn: BlitzGateway, image: ImageWrapper) -> ExternalInfoI:
2416
"""
2517
Get the external info for an OMERO image.
@@ -157,7 +149,7 @@ def set_external_info(
157149
idx: int,
158150
overwrite_path: str,
159151
entityType: str,
160-
entityId: int
152+
entityId: int,
161153
) -> ImageI:
162154
"""
163155
Set the external info for an OMERO image.
@@ -194,12 +186,10 @@ def set_external_info(
194186
else:
195187
if METADATA_XML_RE.match(img_path):
196188
metadata_xml = METADATA_XML_RE.match(img_path).group(1)
197-
path = img_path.replace(metadata_xml,"")
189+
path = img_path.replace(metadata_xml, "")
198190
path = f"/{path}"
199191
else:
200-
raise ValueError(
201-
f"Doesn't seem to be an ome.zarr: {img_path}"
202-
)
192+
raise ValueError(f"Doesn't seem to be an ome.zarr: {img_path}")
203193

204194
if well:
205195
match = WELL_POS_RE.match(well)
@@ -213,7 +203,6 @@ def set_external_info(
213203
series = img.getSeries()._val
214204
path = f"{path}{series}"
215205

216-
217206
info = ExternalInfoI()
218207
info.entityType = rstring(entityType)
219208
info.entityId = rlong(entityId)

0 commit comments

Comments
 (0)