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

implement jointcal constrained model fixes to reduce failing factorizations

    Details

    • Story Points:
      4
    • Sprint:
      Alert Production F17 - 11
    • Team:
      Alert Production

      Description

      Pierre Astier and Dominique Boutigny found a few bugs in jointcal's constrained models that were causing cholmod factorization failures and/or improper fits. I need to implement those changes before doing the meas_mosaic comparison.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            Changes implemented in the PR. Cannot test them on HSC data given the butler problems in DM-12117.

            Jenkins run here: https://ci.lsst.codes/job/stack-os-matrix/27046/

            Show
            Parejkoj John Parejko added a comment - Changes implemented in the PR. Cannot test them on HSC data given the butler problems in DM-12117 . Jenkins run here: https://ci.lsst.codes/job/stack-os-matrix/27046/
            Hide
            Parejkoj John Parejko added a comment -

            Chris Morrison, do you mind reviewing these changes? It's a mix of C++ and python and is a bit of a grab-bag, so you can look at the individual commits to help make sense of the bits.

            Show
            Parejkoj John Parejko added a comment - Chris Morrison , do you mind reviewing these changes? It's a mix of C++ and python and is a bit of a grab-bag, so you can look at the individual commits to help make sense of the bits.
            Hide
            cmorrison Chris Morrison added a comment -

            Finished review. See comments on github.

            Show
            cmorrison Chris Morrison added a comment - Finished review. See comments on github.
            Hide
            Parejkoj John Parejko added a comment -

            Thanks for the review. Merged and done.

            Show
            Parejkoj John Parejko added a comment - Thanks for the review. Merged and done.

              People

              • Assignee:
                Parejkoj John Parejko
                Reporter:
                Parejkoj John Parejko
                Reviewers:
                Chris Morrison
                Watchers:
                Chris Morrison, Dominique Boutigny, John Parejko, John Swinbank, Pierre Astier
              • Votes:
                0 Vote for this issue
                Watchers:
                5 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel