[PIPE2D-63] Clean up genDefectFits.py in obs_pfs Created: 17/Aug/16 Updated: 28/Feb/17 Resolved: 28/Feb/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: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Story Points: | 0.5 |
| Sprint: | 2014-16 |
| Reviewers: | swinbank |
| Description |
|
There are unused imports and some chatter which needs to get cleaned up |
| Comments |
| Comment by swinbank [ 22/Feb/17 ] |
|
Small comment on the code changes on GitHub. In addition, I note that at line 65 the code writes a warning message to sys.stderr. Why not use the logger (at WARN or ERROR level) for that? Once you have thought about the above, good to merge. |
| Comment by aritter [ 28/Feb/17 ] |
|
Merged into master |