Skip to content

Commit 193fcfe

Browse files
TallJimbotimj
authored andcommitted
Add ButlerURI support to config file override paths.
1 parent bf0e57e commit 193fcfe

1 file changed

Lines changed: 7 additions & 4 deletions

File tree

python/lsst/pipe/base/configOverrides.py

Lines changed: 7 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -29,6 +29,7 @@
2929
from enum import Enum
3030
from operator import attrgetter
3131

32+
from lsst.daf.butler import ButlerURI
3233
from lsst.utils import doImport
3334

3435
OverrideTypes = Enum("OverrideTypes", "Value File Python Instrument")
@@ -147,10 +148,11 @@ def addFileOverride(self, filename):
147148
148149
Parameters
149150
----------
150-
filename : str
151-
Path to the override file.
151+
filename : `str` or `ButlerURI`
152+
Path or URI to the override file. All URI schemes supported by
153+
`ButlerURI` are supported.
152154
"""
153-
self._overrides.append((OverrideTypes.File, filename))
155+
self._overrides.append((OverrideTypes.File, ButlerURI(filename)))
154156

155157
def addValueOverride(self, field, value):
156158
"""Add override for a specific field.
@@ -234,7 +236,8 @@ def applyTo(self, config):
234236

235237
for otype, override in self._overrides:
236238
if otype is OverrideTypes.File:
237-
config.load(override)
239+
with override.open("r") as buffer:
240+
config.loadFromStream(buffer, filename=str(override))
238241
elif otype is OverrideTypes.Value:
239242
field, value = override
240243
if isinstance(value, str):

0 commit comments

Comments
 (0)