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

Option to turn sparse matrices into dense ones to explore eigenvalues

    XMLWordPrintable

    Details

    • Story Points:
      2
    • Sprint:
      AP S18-6
    • Team:
      Alert Production

      Description

      I need to explore the properties of the matrix and gradient in jointcal's ConstrainedPhotometyModel, but the sparse matrix code in Eigen doesn't include many options for doing so. To better facilitate this, Pierre Astier suggested outputting the matrix elements and creating a dense matrix from it. This ticket is just for the output and checking that it's sensible: detailed exploration will be in a different ticket.

      A quick search suggests that one can pass the sparse matrix to one of Eigen's constructors to directly get the dense matrix. It might be easier to explore the matrix properties with numpy.

        Attachments

          Issue Links

            Activity

            Hide
            Parejkoj John Parejko added a comment -

            Fred Moolekamp Do you mind taking on this small-ish review? You should be able to review the 4 commits on the PR independently.

            Thank you in advance.

            Show
            Parejkoj John Parejko added a comment - Fred Moolekamp Do you mind taking on this small-ish review? You should be able to review the 4 commits on the PR independently. Thank you in advance.
            Hide
            Parejkoj John Parejko added a comment -

            Thanks for the review. Per your github comment, I agree that the implemented design isn't ideal, but I don't think there's a better way to lift that behavior into python.

            Jim and I are considering some jointcal refactoring, if we ever get the photometry model working, and I'd definitely include rethinking this as part of it.

            Show
            Parejkoj John Parejko added a comment - Thanks for the review. Per your github comment, I agree that the implemented design isn't ideal, but I don't think there's a better way to lift that behavior into python. Jim and I are considering some jointcal refactoring, if we ever get the photometry model working, and I'd definitely include rethinking this as part of it.

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Fred Moolekamp
              Watchers:
              Fred Moolekamp, Jim Bosch, John Parejko, John Swinbank, Pierre Astier
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.