Skip to content

Commit ad3fe93

Browse files
committed
minor
1 parent d2e8215 commit ad3fe93

File tree

9 files changed

+33
-33
lines changed

9 files changed

+33
-33
lines changed

docs/source/contents/datasets.rst

+6-6
Original file line numberDiff line numberDiff line change
@@ -7,28 +7,28 @@ Datasets
77
.. contents::
88
:local:
99

10-
ImagesBaseDataset
10+
ImageBaseDataset
1111
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
12-
.. autoclass:: oml.datasets.images.BaseDataset
12+
.. autoclass:: oml.datasets.images.ImageBaseDataset
1313
:undoc-members:
1414
:show-inheritance:
1515

1616
.. automethod:: __init__
1717
.. automethod:: visualize
1818

19-
ImagesDatasetLabeled
19+
ImageDatasetLabeled
2020
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
21-
.. autoclass:: oml.datasets.images.ImagesDatasetLabeled
21+
.. autoclass:: oml.datasets.images.ImageDatasetLabeled
2222
:undoc-members:
2323
:show-inheritance:
2424

2525
.. automethod:: __init__
2626
.. automethod:: __getitem__
2727
.. automethod:: get_labels
2828

29-
ImagesDatasetQueryGalleryLabeled
29+
ImageDatasetQueryGalleryLabeled
3030
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
31-
.. autoclass:: oml.datasets.images.DatasetQueryGallery
31+
.. autoclass:: oml.datasets.images.ImageDatasetQueryGalleryLabeled
3232
:undoc-members:
3333
:show-inheritance:
3434

oml/datasets/base.py

+5-5
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
1-
from oml.datasets.images import ImagesDatasetLabeled, ImagesDatasetQueryGalleryLabeled
1+
from oml.datasets.images import ImageDatasetLabeled, ImageDatasetQueryGalleryLabeled
22

33

4-
class DatasetWithLabels(ImagesDatasetLabeled):
5-
# this class allows to have back compatibility
4+
class DatasetWithLabels(ImageDatasetLabeled):
5+
# this class allows to have backward compatibility
66
pass
77

88

9-
class DatasetQueryGallery(ImagesDatasetQueryGalleryLabeled):
10-
# this class allows to have back compatibility
9+
class DatasetQueryGallery(ImageDatasetQueryGalleryLabeled):
10+
# this class allows to have backward compatibility
1111
pass
1212

1313

oml/datasets/images.py

+8-8
Original file line numberDiff line numberDiff line change
@@ -75,7 +75,7 @@ def parse_bboxes(df: pd.DataFrame) -> Optional[TBBoxes]:
7575
return bboxes
7676

7777

78-
class ImagesBaseDataset(IBaseDataset, IVisualizableDataset):
78+
class ImageBaseDataset(IBaseDataset, IVisualizableDataset):
7979
"""
8080
The base class that handles image specific logic.
8181
@@ -218,7 +218,7 @@ def bboxes_keys(self) -> Tuple[str, ...]:
218218
return self.x1_key, self.y1_key, self.x2_key, self.y2_key
219219

220220

221-
class ImagesDatasetLabeled(ImagesBaseDataset, IDatasetLabeled):
221+
class ImageDatasetLabeled(ImageBaseDataset, IDatasetLabeled):
222222
"""
223223
The dataset of images having their ground truth labels.
224224
@@ -298,7 +298,7 @@ def get_label2category(self) -> Optional[Dict[int, Union[str, int]]]:
298298
return label2category
299299

300300

301-
class ImagesDatasetQueryGalleryLabeled(ImagesDatasetLabeled, IDatasetQueryGalleryLabeled):
301+
class ImageDatasetQueryGalleryLabeled(ImageDatasetLabeled, IDatasetQueryGalleryLabeled):
302302
"""
303303
The dataset of images having `query`/`gallery` split.
304304
@@ -399,7 +399,7 @@ def get_retrieval_images_datasets(
399399
df_train = df[df[SPLIT_COLUMN] == "train"].reset_index(drop=True)
400400
df_train[LABELS_COLUMN] = df_train[LABELS_COLUMN].map(mapper)
401401

402-
train_dataset = ImagesDatasetLabeled(
402+
train_dataset = ImageDatasetLabeled(
403403
df=df_train,
404404
dataset_root=dataset_root,
405405
transform=transforms_train,
@@ -409,7 +409,7 @@ def get_retrieval_images_datasets(
409409

410410
# val (query + gallery)
411411
df_query_gallery = df[df[SPLIT_COLUMN] == "validation"].reset_index(drop=True)
412-
valid_dataset = ImagesDatasetQueryGalleryLabeled(
412+
valid_dataset = ImageDatasetQueryGalleryLabeled(
413413
df=df_query_gallery,
414414
dataset_root=dataset_root,
415415
transform=transforms_val,
@@ -421,8 +421,8 @@ def get_retrieval_images_datasets(
421421

422422

423423
__all__ = [
424-
"ImagesBaseDataset",
425-
"ImagesDatasetLabeled",
426-
"ImagesDatasetQueryGalleryLabeled",
424+
"ImageBaseDataset",
425+
"ImageDatasetLabeled",
426+
"ImageDatasetQueryGalleryLabeled",
427427
"get_retrieval_images_datasets",
428428
]

oml/datasets/pairs.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
from torch import Tensor
55

66
from oml.const import INDEX_KEY, PAIR_1ST_KEY, PAIR_2ND_KEY, TBBoxes
7-
from oml.datasets.images import ImagesBaseDataset
7+
from oml.datasets.images import ImageBaseDataset
88
from oml.interfaces.datasets import IPairsDataset
99
from oml.transforms.images.torchvision import get_normalisation_torch
1010
from oml.transforms.images.utils import TTransforms
@@ -98,8 +98,8 @@ def __init__(
9898

9999
cache_size = cache_size // 2 if cache_size else None
100100
dataset_args = {"transform": transform, "f_imread": f_imread, "cache_size": cache_size}
101-
self.dataset1 = ImagesBaseDataset(paths=paths1, bboxes=bboxes1, **dataset_args)
102-
self.dataset2 = ImagesBaseDataset(paths=paths2, bboxes=bboxes2, **dataset_args)
101+
self.dataset1 = ImageBaseDataset(paths=paths1, bboxes=bboxes1, **dataset_args)
102+
self.dataset2 = ImageBaseDataset(paths=paths2, bboxes=bboxes2, **dataset_args)
103103

104104
self.pair_1st_key = pair_1st_key
105105
self.pair_2nd_key = pair_2nd_key

oml/inference/flat.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from torch import Tensor, nn
88

99
from oml.const import PATHS_COLUMN, SPLIT_COLUMN
10-
from oml.datasets.images import ImagesBaseDataset
10+
from oml.datasets.images import ImageBaseDataset
1111
from oml.inference.abstract import _inference
1212
from oml.interfaces.models import IExtractor
1313
from oml.transforms.images.utils import TTransforms
@@ -28,7 +28,7 @@ def inference_on_images(
2828
use_fp16: bool = False,
2929
accumulate_on_cpu: bool = True,
3030
) -> Tensor:
31-
dataset = ImagesBaseDataset(paths=paths, bboxes=None, transform=transform, f_imread=f_imread, cache_size=0)
31+
dataset = ImageBaseDataset(paths=paths, bboxes=None, transform=transform, f_imread=f_imread, cache_size=0)
3232
device = get_device(model)
3333

3434
def _apply(model_: nn.Module, batch_: Dict[str, Any]) -> Tensor:

oml/interfaces/datasets.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -74,7 +74,7 @@ def get_gallery_ids(self) -> LongTensor:
7474

7575
class IDatasetQueryGalleryLabeled(IDatasetQueryGallery, IDatasetLabeled, ABC):
7676
"""
77-
This class is similar to "IDatasetQueryGalleryPrediction", but we also have ground truth labels.
77+
This interface is similar to `IDatasetQueryGallery`, but there are ground truth labels.
7878
"""
7979

8080

oml/lightning/pipelines/predict.py

+2-2
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@
77
from torch.utils.data import DataLoader
88

99
from oml.const import IMAGE_EXTENSIONS, TCfg
10-
from oml.datasets.images import ImagesBaseDataset
10+
from oml.datasets.images import ImageBaseDataset
1111
from oml.ddp.utils import get_world_size_safe, is_main_process, sync_dicts_ddp
1212
from oml.lightning.modules.extractor import ExtractorModule
1313
from oml.lightning.pipelines.parser import parse_engine_params_from_config
@@ -41,7 +41,7 @@ def extractor_prediction_pipeline(cfg: TCfg) -> None:
4141
if broken_images:
4242
raise ValueError(f"There are images that cannot be open:\n {broken_images}.")
4343

44-
dataset = ImagesBaseDataset(paths=filenames, transform=transforms, f_imread=f_imread)
44+
dataset = ImageBaseDataset(paths=filenames, transform=transforms, f_imread=f_imread)
4545

4646
loader = DataLoader(
4747
dataset=dataset, batch_size=cfg["bs"], num_workers=cfg["num_workers"], shuffle=False, drop_last=False

oml/lightning/pipelines/train_postprocessor.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
from torch.utils.data import DataLoader
1212

1313
from oml.const import BBOXES_COLUMNS, EMBEDDINGS_KEY, TCfg
14-
from oml.datasets.base import ImagesDatasetLabeled, ImagesDatasetQueryGalleryLabeled
14+
from oml.datasets.base import ImageDatasetLabeled, ImageDatasetQueryGalleryLabeled
1515
from oml.inference.flat import inference_on_dataframe
1616
from oml.interfaces.models import IPairwiseModel
1717
from oml.lightning.callbacks.metric import MetricValCallback, MetricValCallbackDDP
@@ -81,13 +81,13 @@ def get_loaders_with_embeddings(cfg: TCfg) -> Tuple[DataLoader, DataLoader]:
8181
use_fp16=int(cfg.get("precision", 32)) == 16,
8282
)
8383

84-
train_dataset = ImagesDatasetLabeled(
84+
train_dataset = ImageDatasetLabeled(
8585
df=df_train,
8686
transform=get_transforms_by_cfg(cfg["transforms_train"]),
8787
extra_data={EMBEDDINGS_KEY: emb_train},
8888
)
8989

90-
valid_dataset = ImagesDatasetQueryGalleryLabeled(
90+
valid_dataset = ImageDatasetQueryGalleryLabeled(
9191
df=df_val,
9292
# we don't care about transforms, since the only goal of this dataset is to deliver embeddings
9393
transform=get_normalisation_resize_torch(im_size=8),

tests/test_oml/test_transforms/test_image_augs.py

+3-3
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
from omegaconf import OmegaConf
66

77
from oml.const import CONFIGS_PATH, MOCK_DATASET_PATH
8-
from oml.datasets.images import ImagesDatasetLabeled
8+
from oml.datasets.images import ImageDatasetLabeled
99
from oml.registry.transforms import TRANSFORMS_REGISTRY, get_transforms_by_cfg
1010

1111

@@ -14,7 +14,7 @@ def test_transforms(aug_name: Optional[str]) -> None:
1414
df = pd.read_csv(MOCK_DATASET_PATH / "df.csv")
1515
transforms = get_transforms_by_cfg(OmegaConf.load(CONFIGS_PATH / "transforms" / f"{aug_name}.yaml"))
1616

17-
dataset = ImagesDatasetLabeled(df=df, dataset_root=MOCK_DATASET_PATH, transform=transforms)
17+
dataset = ImageDatasetLabeled(df=df, dataset_root=MOCK_DATASET_PATH, transform=transforms)
1818

1919
_ = dataset[0]
2020

@@ -23,6 +23,6 @@ def test_transforms(aug_name: Optional[str]) -> None:
2323

2424
def test_default_transforms() -> None:
2525
df = pd.read_csv(MOCK_DATASET_PATH / "df.csv")
26-
dataset = ImagesDatasetLabeled(df=df, dataset_root=MOCK_DATASET_PATH, transform=None)
26+
dataset = ImageDatasetLabeled(df=df, dataset_root=MOCK_DATASET_PATH, transform=None)
2727
_ = dataset[0]
2828
assert True

0 commit comments

Comments
 (0)