Skip to content

Commit

Permalink
Change non-MeasurementErrors to warnings
Browse files Browse the repository at this point in the history
This corrects a miscommunication on DM-9574, which was supposed to be about
adding new DEBUG logs for other errors, but which was applied too broadly.
These exceptions tell us about important failure modes that we should fix, so
we shouldn't be hiding them in the debug logs.
  • Loading branch information
parejkoj committed Mar 26, 2024
1 parent 876c692 commit bddf5a5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions python/lsst/meas/base/baseMeasurement.py
Original file line number Diff line number Diff line change
Expand Up @@ -393,7 +393,7 @@ def doMeasurement(self, plugin, measRecord, *args, **kwds):
plugin.name, measRecord.getId(), error)
plugin.fail(measRecord, error)
except Exception as error:
self.log.getChild(plugin.name).debug(
self.log.getChild(plugin.name).warning(
"Exception in %s.measure on record %s: %s",
plugin.name, measRecord.getId(), error)
plugin.fail(measRecord)
Expand Down Expand Up @@ -482,6 +482,6 @@ def doMeasurementN(self, plugin, measCat, *args, **kwds):
except Exception as error:
for measRecord in measCat:
plugin.fail(measRecord)
self.log.getChild(plugin.name).debug(
self.log.getChild(plugin.name).warning(
"Exception in %s.measureN on records %s-%s: %s",
plugin.name, measCat[0].getId(), measCat[-1].getId(), error)

0 comments on commit bddf5a5

Please sign in to comment.