Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-2304

Internally cite the Bernstein and Jarvis (2002) paper in the SDSS image moment shape algorithm code

    Details

    • Type: Improvement
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: meas_base
    • Labels:
      None
    • Team:
      Data Release Production

      Description

      The algorithm to "Measure the image moments of source using adaptive Gaussian weights" implemented in meas_base should reference the paper from which the algorithm is derived (Bernstein and Jarvis (2002) )

      The reference is here:

      http://adsabs.harvard.edu/abs/2002AJ....123..583B

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            I wasn't actually sure if BJ02 was the first place this algorithm was derived, or if actually was the inspiration for the original SdssShape. I think of BJ02 as being more about how to go from those sort of moments to PSF-corrected ellipticities. Robert Lupton might be able to say more about the algorithm provenance here.

            Show
            jbosch Jim Bosch added a comment - I wasn't actually sure if BJ02 was the first place this algorithm was derived, or if actually was the inspiration for the original SdssShape. I think of BJ02 as being more about how to go from those sort of moments to PSF-corrected ellipticities. Robert Lupton might be able to say more about the algorithm provenance here.
            Hide
            swinbank John Swinbank added a comment -

            Relevant discussion on HipChat:

            K-T Lim: I'm pretty sure it's Bernstein and Jarvis.
            [...]
            Robert Lupton: @ChrisWalter @ktl's right, it's http://www.journals.uchicago.edu/AJ/journal/issues/v123n2/201306/201306.html
            Robert Lupton: FIle an issue to add it to the docs

            Show
            swinbank John Swinbank added a comment - Relevant discussion on HipChat: K-T Lim: I'm pretty sure it's Bernstein and Jarvis. [...] Robert Lupton: @ChrisWalter @ktl's right, it's http://www.journals.uchicago.edu/AJ/journal/issues/v123n2/201306/201306.html Robert Lupton: FIle an issue to add it to the docs
            Hide
            cwalter Chris Walter added a comment -

            Note, make sure to use my reference in the summary above. Robert's no longer works.

            Show
            cwalter Chris Walter added a comment - Note, make sure to use my reference in the summary above. Robert's no longer works.
            Hide
            swinbank John Swinbank added a comment -

            The work will be done on DM-1161.

            Show
            swinbank John Swinbank added a comment - The work will be done on DM-1161 .
            Hide
            cwalter Chris Walter added a comment -

            Just for my own jira knowledge: I was trying to track how this is related to DM-1161. But it is marked here as "Won't fix" and that is how it shows there also. Is that because there is nothing that better maps to "assigned to another issue"?

            Show
            cwalter Chris Walter added a comment - Just for my own jira knowledge: I was trying to track how this is related to DM-1161 . But it is marked here as "Won't fix" and that is how it shows there also. Is that because there is nothing that better maps to "assigned to another issue"?
            Hide
            jbosch Jim Bosch added a comment -

            Correct. At some point someone was going to try to create a "Duplicate" status that would be more appropriate, but I think that effort stalled, and I've been spending too much of my energy on RFCs to try pick it up myself.

            Show
            jbosch Jim Bosch added a comment - Correct. At some point someone was going to try to create a "Duplicate" status that would be more appropriate, but I think that effort stalled, and I've been spending too much of my energy on RFCs to try pick it up myself.
            Hide
            swinbank John Swinbank added a comment -

            In fact, we should really note specifically at DM-1161 that this is part of the requirements. Jim & I rushed through a bunch of issues earlier and it was my plan to come back and tidy up later. Better do that now...

            Show
            swinbank John Swinbank added a comment - In fact, we should really note specifically at DM-1161 that this is part of the requirements. Jim & I rushed through a bunch of issues earlier and it was my plan to come back and tidy up later. Better do that now...

              People

              • Assignee:
                jbosch Jim Bosch
                Reporter:
                cwalter Chris Walter
                Watchers:
                Chris Walter, Jim Bosch, John Swinbank
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel