[PIPE2D-296] Add plotting capability for SpectrumSet Created: 18/Oct/18 Updated: 19/Oct/18 Resolved: 18/Oct/18 |
|
| Status: | Done |
| Project: | DRP 2-D Pipeline |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | None |
| Type: | Story | Priority: | Normal |
| Reporter: | price | Assignee: | price |
| Resolution: | Done | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
| Reviewers: | hassan |
| Description |
|
It would be helpful when debugging to be able to easily plot a SpectrumSet. |
| Comments |
| Comment by price [ 18/Oct/18 ] |
|
hassan, would you be willing to review this modest addition? |
| Comment by price [ 18/Oct/18 ] |
|
Merged to master after all comments on GitHub PR were resolved. |
| Comment by rhl [ 19/Oct/18 ] |
|
How does this relate to the plotting code enabled by using lsstDebug?
|
| Comment by price [ 19/Oct/18 ] |
|
This is general-purpose plotting code that can be called as spectrum.plot(). I don't think it is specifically enabled anywhere in the pipeline (apart from tests) at the moment, including lsstDebug blocks; I use it on the command-line to verify if something is working. |
| Comment by rhl [ 19/Oct/18 ] |
|
There is also generalish purpose code enabled by debug options. We should probably make it available from the command line (or really notebook), and if there is overlap it should be resolved |