-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[Feature] Add a tool to find broken files. (#482)
* add verify dataset * add phase * rm attr of single_process * Use `mmcv.track_parallel_progress` to track the validation. Co-authored-by: mzr1996 <mzr1996@163.com>
- Loading branch information
Showing
1 changed file
with
131 additions
and
0 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,131 @@ | ||
# Copyright (c) OpenMMLab. All rights reserved. | ||
import argparse | ||
import fcntl | ||
import os | ||
from pathlib import Path | ||
|
||
from mmcv import Config, DictAction, track_parallel_progress, track_progress | ||
|
||
from mmcls.datasets import PIPELINES, build_dataset | ||
|
||
|
||
def parse_args(): | ||
parser = argparse.ArgumentParser(description='Verify Dataset') | ||
parser.add_argument('config', help='config file path') | ||
parser.add_argument( | ||
'--out-path', | ||
type=str, | ||
default='brokenfiles.log', | ||
help='output path of all the broken files. If the specified path ' | ||
'already exists, delete the previous file ') | ||
parser.add_argument( | ||
'--phase', | ||
default='train', | ||
type=str, | ||
choices=['train', 'test', 'val'], | ||
help='phase of dataset to visualize, accept "train" "test" and "val".') | ||
parser.add_argument( | ||
'--num-process', type=int, default=1, help='number of process to use') | ||
parser.add_argument( | ||
'--cfg-options', | ||
nargs='+', | ||
action=DictAction, | ||
help='override some settings in the used config, the key-value pair ' | ||
'in xxx=yyy format will be merged into config file. If the value to ' | ||
'be overwritten is a list, it should be like key="[a,b]" or key=a,b ' | ||
'It also allows nested list/tuple values, e.g. key="[(a,b),(c,d)]" ' | ||
'Note that the quotation marks are necessary and that no white space ' | ||
'is allowed.') | ||
args = parser.parse_args() | ||
assert args.out_path is not None | ||
assert args.num_process > 0 | ||
return args | ||
|
||
|
||
class DatasetValidator(): | ||
"""the dataset tool class to check if all file are broken.""" | ||
|
||
def __init__(self, dataset_cfg, log_file_path, phase): | ||
super(DatasetValidator, self).__init__() | ||
# keep only LoadImageFromFile pipeline | ||
assert dataset_cfg.data[phase].pipeline[0][ | ||
'type'] == 'LoadImageFromFile', 'This tool is only for dataset ' \ | ||
'that needs to load image from files.' | ||
self.pipeline = PIPELINES.build(dataset_cfg.data[phase].pipeline[0]) | ||
dataset_cfg.data[phase].pipeline = [] | ||
dataset = build_dataset(dataset_cfg.data[phase]) | ||
|
||
self.dataset = dataset | ||
self.log_file_path = log_file_path | ||
|
||
def valid_idx(self, idx): | ||
item = self.dataset[idx] | ||
try: | ||
item = self.pipeline(item) | ||
except Exception: | ||
with open(self.log_file_path, 'a') as f: | ||
# add file lock to prevent multi-process writing errors | ||
fcntl.flock(f.fileno(), fcntl.LOCK_EX) | ||
filepath = os.path.join(item['img_prefix'], | ||
item['img_info']['filename']) | ||
f.write(filepath + '\n') | ||
print(f'{filepath} cannot be read correctly, please check it.') | ||
# Release files lock automatic using with | ||
|
||
def __len__(self): | ||
return len(self.dataset) | ||
|
||
|
||
def print_info(log_file_path): | ||
"""print some information and do extra action.""" | ||
print() | ||
with open(log_file_path, 'r') as f: | ||
context = f.read().strip() | ||
if context == '': | ||
print('There is no broken file found.') | ||
os.remove(log_file_path) | ||
else: | ||
num_file = len(context.split('\n')) | ||
print(f'{num_file} broken files found, name list save in file:' | ||
f'{log_file_path}') | ||
print() | ||
|
||
|
||
def main(): | ||
# parse cfg and args | ||
args = parse_args() | ||
cfg = Config.fromfile(args.config) | ||
if args.cfg_options is not None: | ||
cfg.merge_from_dict(args.cfg_options) | ||
|
||
# touch output file to save broken files list. | ||
output_path = Path(args.out_path) | ||
if not output_path.parent.exists(): | ||
raise Exception('log_file parent directory not found.') | ||
if output_path.exists(): | ||
os.remove(output_path) | ||
output_path.touch() | ||
|
||
# do valid | ||
validator = DatasetValidator(cfg, output_path, args.phase) | ||
|
||
if args.num_process > 1: | ||
# The default chunksize calcuation method of Pool.map | ||
chunksize, extra = divmod(len(validator), args.num_process * 8) | ||
if extra: | ||
chunksize += 1 | ||
|
||
track_parallel_progress( | ||
validator.valid_idx, | ||
list(range(len(validator))), | ||
args.num_process, | ||
chunksize=chunksize, | ||
keep_order=False) | ||
else: | ||
track_progress(validator.valid_idx, list(range(len(validator)))) | ||
|
||
print_info(output_path) | ||
|
||
|
||
if __name__ == '__main__': | ||
main() |