Stop using deprecated Butler constructor

Directly calling the Butler() constructor has been deprecated because it is not compatible with mypy. This has been replaced with Butler.from_config().
lsst-ts · Mar 4, 2024 · efed055 · efed055
1 parent 2d16294
commit efed055
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/tests/data/print_registry_contents.py b/tests/data/print_registry_contents.py
@@ -11,7 +11,7 @@ def print_registry_contents(name: str) -> None:
         The instrument name, which must match the registry name, e.g. "LATISS".
     """
     print(f"\nregistry=instrument={name}\n")
-    butler = lsst.daf.butler.Butler(name, writeable=False)
+    butler = lsst.daf.butler.Butler.from_config(name, writeable=False)
     registry = butler.registry
     record_iter = registry.queryDimensionRecords(
         "exposure",

diff --git a/tests/test_find_exposures.py b/tests/test_find_exposures.py
@@ -108,7 +108,7 @@ async def test_find_exposures_one_registry(self) -> None:
         repo_path = pathlib.Path(__file__).parent / "data" / instrument
 
         # Find all exposures in the registry, and save as a list of dicts.
-        butler = lsst.daf.butler.Butler(str(repo_path))
+        butler = lsst.daf.butler.Butler.from_config(str(repo_path))
         registry = butler.registry
         exposure_iter = registry.queryDimensionRecords(
             "exposure",
@@ -395,7 +395,7 @@ async def test_find_exposures_two_registries(self) -> None:
         # thus searches only return exposures from one registry.
         # Use instrument=LATISS to search the second registry
         # in order to test DM-33601.
-        butler = lsst.daf.butler.Butler(str(repo_path_2))
+        butler = lsst.daf.butler.Butler.from_config(str(repo_path_2))
         registry = butler.registry
         exposure_iter = registry.queryDimensionRecords(
             "exposure",