From 5840e528cac1d64a4a0a08e7da6fd773a66c0c2f Mon Sep 17 00:00:00 2001 From: Yanghan Wang Date: Sun, 11 Jun 2023 22:12:43 -0700 Subject: [PATCH] fix d2go.config Summary: I think the main issue is that we import `reroute_config_path` from `d2go.config.config` in `__init__.py`, but it's actually in `d2go.config.utils`. After fixing this, the namespace forward also works, see `scripts/wangyanghan/autodeps_testbed/d2go_config/TARGETS` Update all TARGETS: ``` fbgs -l "d2go/config:" | xargs printf -- '/data/sandcastle/boxes/%s\n' | xargs arc lint -a ``` For reviewers, only `.autodeps.toml` and files in `d2go/d2go/config/` and `scripts/wangyanghan/autodeps_testbed/d2go_config/` are manually changed, other files are auto modified. Reviewed By: ajinkya-deogade Differential Revision: D46582416 fbshipit-source-id: c801d47717e3297700a7bbd52f974653ee760826 --- d2go/config/__init__.py | 2 +- d2go/config/config.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/d2go/config/__init__.py b/d2go/config/__init__.py index 5ea4a896..f5b00163 100644 --- a/d2go/config/__init__.py +++ b/d2go/config/__init__.py @@ -9,10 +9,10 @@ CONFIG_CUSTOM_PARSE_REGISTRY, CONFIG_SCALING_METHOD_REGISTRY, load_full_config_from_file, - reroute_config_path, temp_defrost, temp_new_allowed, ) +from d2go.config.utils import reroute_config_path __all__ = [ diff --git a/d2go/config/config.py b/d2go/config/config.py index 1175d8e3..39020176 100644 --- a/d2go/config/config.py +++ b/d2go/config/config.py @@ -5,8 +5,8 @@ import copy import logging from typing import List +from unittest import mock -import mock import yaml from d2go.config.utils import reroute_config_path, resolve_default_config from detectron2.config import CfgNode as _CfgNode