Skip to content

Commit 13ada56

Browse files
youcefouadjerYoucef OUADJERNicolasHug
authored
Raise error when download=True for LFW dataset #8888 (#9040)
Co-authored-by: Youcef OUADJER <[email protected]> Co-authored-by: Nicolas Hug <[email protected]> Co-authored-by: Nicolas Hug <[email protected]>
1 parent 72b9d62 commit 13ada56

File tree

1 file changed

+16
-6
lines changed

1 file changed

+16
-6
lines changed

torchvision/datasets/lfw.py

Lines changed: 16 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -51,6 +51,10 @@ def __init__(
5151
self.data: list[Any] = []
5252

5353
if download:
54+
raise ValueError(
55+
"LFW dataset is no longer available for download."
56+
"Please download the dataset manually and place it in the specified directory"
57+
)
5458
self.download()
5559

5660
if not self._check_integrity():
@@ -90,6 +94,11 @@ def __len__(self) -> int:
9094
class LFWPeople(_LFW):
9195
"""`LFW <http://vis-www.cs.umass.edu/lfw/>`_ Dataset.
9296
97+
.. warning:
98+
99+
The LFW dataset is no longer available for automatic download. Please
100+
download it manually and place it in the specified directory.
101+
93102
Args:
94103
root (str or ``pathlib.Path``): Root directory of dataset where directory
95104
``lfw-py`` exists or will be saved to if download is set to True.
@@ -101,9 +110,7 @@ class LFWPeople(_LFW):
101110
and returns a transformed version. E.g, ``transforms.RandomCrop``
102111
target_transform (callable, optional): A function/transform that takes in the
103112
target and transforms it.
104-
download (bool, optional): If true, downloads the dataset from the internet and
105-
puts it in root directory. If dataset is already downloaded, it is not
106-
downloaded again.
113+
download (bool, optional): NOT SUPPORTED ANYMORE, leave to False.
107114
loader (callable, optional): A function to load an image given its path.
108115
By default, it uses PIL as its image loader, but users could also pass in
109116
``torchvision.io.decode_image`` for decoding image data into tensors directly.
@@ -175,6 +182,11 @@ def extra_repr(self) -> str:
175182
class LFWPairs(_LFW):
176183
"""`LFW <http://vis-www.cs.umass.edu/lfw/>`_ Dataset.
177184
185+
.. warning:
186+
187+
The LFW dataset is no longer available for automatic download. Please
188+
download it manually and place it in the specified directory.
189+
178190
Args:
179191
root (str or ``pathlib.Path``): Root directory of dataset where directory
180192
``lfw-py`` exists or will be saved to if download is set to True.
@@ -186,9 +198,7 @@ class LFWPairs(_LFW):
186198
and returns a transformed version. E.g, ``transforms.RandomRotation``
187199
target_transform (callable, optional): A function/transform that takes in the
188200
target and transforms it.
189-
download (bool, optional): If true, downloads the dataset from the internet and
190-
puts it in root directory. If dataset is already downloaded, it is not
191-
downloaded again.
201+
download (bool, optional): NOT SUPPORTED ANYMORE, leave to False.
192202
loader (callable, optional): A function to load an image given its path.
193203
By default, it uses PIL as its image loader, but users could also pass in
194204
``torchvision.io.decode_image`` for decoding image data into tensors directly.

0 commit comments

Comments
 (0)