Add stacklevel parameter to the warnings

timj · timj · commit 6fb36c3d234c · 2023-07-28T16:11:11.000-07:00
This makes them come from the code that called the run method
with expIdBits.
diff --git a/python/lsst/ap/association/diaForcedSource.py b/python/lsst/ap/association/diaForcedSource.py
@@ -135,6 +135,7 @@ def run(self,
             warnings.warn(
                 "'expIdBits' argument is deprecated in favor of 'idGenerator'; will be removed after v26.",
                 category=FutureWarning,
+                stacklevel=3,  # Caller + timeMethod
             )
 
         if idGenerator is None:
diff --git a/python/lsst/ap/association/packageAlerts.py b/python/lsst/ap/association/packageAlerts.py
@@ -119,6 +119,7 @@ def run(self,
             warnings.warn(
                 "The 'ccdExposureIdBits' argument is deprecated and unused; it will be removed after v26.",
                 category=FutureWarning,
+                stacklevel=3,  # Caller + timeMethod
             )
         alerts = []
         self._patchDiaSources(diaSourceCat)

Original file line number	Diff line number	Diff line change
`@@ -135,6 +135,7 @@ def run(self,`
`135`	`135`	`warnings.warn(`
`136`	`136`	`"'expIdBits' argument is deprecated in favor of 'idGenerator'; will be removed after v26.",`
`137`	`137`	`category=FutureWarning,`
	`138`	`+ stacklevel=3, # Caller + timeMethod`
`138`	`139`	`)`
`139`	`140`
`140`	`141`	`if idGenerator is None:`
Original file line number	Diff line number	Diff line change
`@@ -119,6 +119,7 @@ def run(self,`
`119`	`119`	`warnings.warn(`
`120`	`120`	`"The 'ccdExposureIdBits' argument is deprecated and unused; it will be removed after v26.",`
`121`	`121`	`category=FutureWarning,`
	`122`	`+ stacklevel=3, # Caller + timeMethod`
`122`	`123`	`)`
`123`	`124`	`alerts = []`
`124`	`125`	`self._patchDiaSources(diaSourceCat)`