[#PIPE2D-70] In ingest.py replace print statements with debug info

[PIPE2D-70] In ingest.py replace print statements with debug info Created: 18/Aug/16 Updated: 25/Mar/17 Resolved: 25/Mar/17
Status:	Done
Project:	DRP 2-D Pipeline
Component/s:	None
Affects Version/s:	None
Fix Version/s:	None

Type:

Story

Priority:

Major

Reporter:

aritter

Assignee:

aritter

Resolution:

Done

Votes:

Labels:

None

Remaining Estimate:

Not Specified

Time Spent:

Not Specified

Original Estimate:

Not Specified

Story Points:

0.1

Sprint:

2014-16

Reviewers:

cloomis

Description

In obs_pfs/python/lsst/obs/pfs/ingest.py replace print statements with debug info

Comments

Comment by aritter [ 18/Aug/16 ]

see tickets/~~PIPE2D-70~~ in obs_pfs

Comment by swinbank [ 31/Jan/17 ]

Hey cloomis: this came up at today's meeting. Do you have an ETA on a review?

Comment by cloomis [ 08/Feb/17 ]

Good news: it still rebases cleanly from master!

But two things:

Shouldn't self.log.logdebug() be self.log.debug()? Or did lsst.log.Log add logdebug at some point?
There are (only) a few more print statements in obs_pfs: in bin.src{genCalibRegistry.py, bin.src/genDefectFits.py, and {tests/PfsFiberTrace.py}}. Should they be fixed in this ticket?

Comment by cloomis [ 25/Mar/17 ]

Comments addressed. Review done.

Comment by aritter [ 25/Mar/17 ]

Addressed both points and pushed to git, pull request initiated

Comment by aritter [ 25/Mar/17 ]

Merged into master

Generated at Sat Feb 10 15:47:57 JST 2024 using Jira 8.3.4#803005-sha1:1f96e09b3c60279a408a2ae47be3c745f571388b.