Skip to content

Commit

Permalink
Skip loading base/config.yml on Windows. (#497)
Browse files Browse the repository at this point in the history
* compose path with os.path.join for base/config.yml

This is a necessary change to work on Windows which the path separator is `\`.
  • Loading branch information
koron authored Apr 2, 2024
1 parent 6e17b5c commit 61ec75f
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion ann_benchmarks/definitions.py
Original file line number Diff line number Diff line change
Expand Up @@ -127,7 +127,7 @@ def get_config_files(base_dir: str = "ann_benchmarks/algorithms") -> List[str]:
"""Get config files for all algorithms."""
config_files = glob.glob(os.path.join(base_dir, "*", "config.yml"))
return list(
set(config_files) - {f"{base_dir}/base/config.yml"}
set(config_files) - {os.path.join(base_dir, "base", "config.yml")}
)

def load_configs(point_type: str, base_dir: str = "ann_benchmarks/algorithms") -> Dict[str, Any]:
Expand Down

0 comments on commit 61ec75f

Please sign in to comment.