From 01c8f60280a0718ca0a12924b91f360c19c55184 Mon Sep 17 00:00:00 2001 From: Tohrusky <65994850+Tohrusky@users.noreply.github.com> Date: Sat, 27 Sep 2025 06:10:42 +0800 Subject: [PATCH] fix: replace KeyError with warning for duplicate registration --- cccv/util/registry.py | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/cccv/util/registry.py b/cccv/util/registry.py index ff6c3a9..9b4a229 100644 --- a/cccv/util/registry.py +++ b/cccv/util/registry.py @@ -1,7 +1,5 @@ # Copyright (c) Facebook, Inc. and its affiliates. All Rights Reserved - -# pyre-strict -# pyre-ignore-all-errors[2,3] +import warnings from typing import Any, Dict, Iterable, Iterator, Optional, Tuple @@ -41,7 +39,11 @@ def __init__(self, name: str) -> None: def _do_register(self, name: str, obj: Any) -> None: if name in self._obj_map: - raise KeyError(f"[CCCV] An object named '{name}' was already registered in '{self._name}' registry!") + warnings.warn( + f"[CCCV] An object named '{name}' was already registered in '{self._name}' registry! We will NOT overwrite the existing one.", + UserWarning, + stacklevel=2, + ) else: self._obj_map[name] = obj