Lee Kelvin added a comment - 18/Jul/22 9:54 PM A new SkyObjectSkyPlot class was added in a new file: ' python/lsst/analysis/tools/analysisPlots/skyObject.py '. As part of this ticket, a small issue with the 'band' naming convention was fixed in the 'Ap12_PSF_skyPlot' class (from which this sky object class was originally based).

Lee Kelvin added a comment - 18/Jul/22 9:58 PM - edited An example Python snippet which tests the above new code functionality is reproduced here, for reference (output plot attached to ticket): collection = "HSC/runs/RC2/w_2022_16/DM-34451" dataId = { "instrument" : "HSC" , "skymap" : "hsc_rings_v1" , "tract" : 9813 }   butler = dafButler.Butler( "/repo/main/" ) objTable = butler.get( "objectTable_tract" , collections = collection, dataId = dataId)   # import pdb; pdb.set_trace()   skyPlot = SkyObjectSkyPlot() skyPlot.populatePrepFromProcess() stage1 = skyPlot.prep(objTable, band = "i" ) stage2 = skyPlot.process(stage1, band = "i" ) plot = skyPlot.post_process(stage2, plotInfo = _StandinPlotInfo())   plt.show()   plt.savefig( "skyObjectSkyTest.png" ) plt.close() print (plot)

Sophie Reed added a comment - 19/Jul/22 2:37 PM Looks good to me, as long as both plots are tested and look good you are good to merge.

Lee Kelvin added a comment - 19/Jul/22 2:54 PM Currently existing plot output also added to this ticket as generated by this ticket branch, for reference. Looks ok to me I think.

Lee Kelvin added a comment - 19/Jul/22 3:17 PM As discussed, additional minor edits to sync up the axis labels between both plots, adding units (degrees). Thanks for the quick review - branch merged and deleted!