From eb96fc6321f0b939171853fefbf5fe87cc6482d3 Mon Sep 17 00:00:00 2001 From: Tamas Bela Feher Date: Wed, 18 Oct 2023 21:17:33 +0200 Subject: [PATCH] Remove unnecessary dataset path check in ANN bench (#1908) This PR removes a superfluous path check. This check is not necessary, since the actual dataset path is computed differently (see `legacy_result_folder`), using the path field in the config file. The existing test ties the dataset file name to the config file name, therefore imposes an unnecessary restriction. In practice I often have multiple json configs using the same dataset dir. Authors: - Tamas Bela Feher (https://github.com/tfeher) Approvers: - Ben Frederickson (https://github.com/benfred) URL: https://github.com/rapidsai/raft/pull/1908 --- python/raft-ann-bench/src/raft-ann-bench/run/__main__.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/python/raft-ann-bench/src/raft-ann-bench/run/__main__.py b/python/raft-ann-bench/src/raft-ann-bench/run/__main__.py index 9e2da7328d..3b670dc464 100644 --- a/python/raft-ann-bench/src/raft-ann-bench/run/__main__.py +++ b/python/raft-ann-bench/src/raft-ann-bench/run/__main__.py @@ -243,8 +243,6 @@ def main(): dataset_path = args.dataset_path if not os.path.exists(conf_filepath): raise FileNotFoundError(conf_filename) - if not os.path.exists(os.path.join(args.dataset_path, dataset_name)): - raise FileNotFoundError(os.path.join(args.dataset_path, dataset_name)) with open(conf_filepath, "r") as f: conf_file = json.load(f)