Skip to content

Commit cb77aa3

Browse files
vincentqbfacebook-github-bot
authored andcommitted
[fbsync] Add explanation for injected fake data in dataset tests (#3576)
Reviewed By: fmassa Differential Revision: D27127996 fbshipit-source-id: 64a9bbf801010555b0fa91ea72b8ff158c7260e1
1 parent 7702df2 commit cb77aa3

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

test/datasets_utils.py

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -266,6 +266,10 @@ def dataset_args(self, tmpdir: str, config: Dict[str, Any]) -> Sequence[Any]:
266266
def inject_fake_data(self, tmpdir: str, config: Dict[str, Any]) -> Union[int, Dict[str, Any]]:
267267
"""Inject fake data for dataset into a temporary directory.
268268
269+
During the creation of the dataset the download and extract logic is disabled. Thus, the fake data injected
270+
here needs to resemble the raw data, i.e. the state of the dataset directly after the files are downloaded and
271+
potentially extracted.
272+
269273
Args:
270274
tmpdir (str): Path to a temporary directory. For most cases this acts as root directory for the dataset
271275
to be created and in turn also for the fake data injected here.

0 commit comments

Comments
 (0)