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

SourceTable should use LocalNanojansky functor on apFluxes

    XMLWordPrintable

    Details

    • Story Points:
      1
    • Epic Link:
    • Team:
      Data Release Production
    • Urgent?:
      No

      Description

      We added a bunch of aperture fluxes in Sept with the wrong flux conversion functor. Should use LocalNanoJansky for all

        Attachments

          Activity

          Ascending order - Click to sort in descending order
          Hide
          Permalink
          yusra Yusra AlSayyad added a comment -

          Thanks Lee, Jenkins going on: https://ci.lsst.codes/job/stack-os-matrix/35239/display/redirect

          Show
          yusra Yusra AlSayyad added a comment - Thanks Lee, Jenkins going on: https://ci.lsst.codes/job/stack-os-matrix/35239/display/redirect
          Hide
          Permalink
          lskelvin Lee Kelvin added a comment -

          Thanks for spotting and addressing this Yusra, this is great. As far as I can tell, everything looks good here. All fluxes are now attributed to the LocalNanojansky functor (with the correct lower-case J for 'jansky'), all flux errors are now attributed to the LocalNanojanskyErr functor, and all added arguments correctly point to their respective pixel radius columns. Assuming Jenkins doesn't complain, then this looks good to merge to me - thanks!

          Show
          lskelvin Lee Kelvin added a comment - Thanks for spotting and addressing this Yusra, this is great. As far as I can tell, everything looks good here. All fluxes are now attributed to the LocalNanojansky functor (with the correct lower-case J for 'jansky'), all flux errors are now attributed to the LocalNanojanskyErr functor, and all added arguments correctly point to their respective pixel radius columns. Assuming Jenkins doesn't complain, then this looks good to merge to me - thanks!
          Hide
          Permalink
          yusra Yusra AlSayyad added a comment -

          Thank you Lee! Merged.

          Show
          yusra Yusra AlSayyad added a comment - Thank you Lee! Merged.

            People

            Assignee:
            yusra Yusra AlSayyad
            Reporter:
            yusra Yusra AlSayyad
            Reviewers:
            Lee Kelvin
            Watchers:
            Lee Kelvin, Yusra AlSayyad
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved:

                Jenkins

                No builds found.