search

jennykim1016 / SLRealizer

Catalog-level simulation of LSST DM stack measurements of gravitationally-lensed quasars.
BSD 3-Clause "New" or "Revised" License
1 stars 0 forks source link

covariance matrix error #21

Closed jennykim1016 closed 7 years ago

jennykim1016 commented 7 years ago
screen shot 2017-07-24 at 1 29 37 pm

If I divide by m (which is a total flux), the values are not reasonable. However, if I do not divide by m, then it gives reasonable values. I think it is related to multiplying multivariate gaussian later, but I am not entirely sure.

jennykim1016 commented 7 years ago

Fixed!

drphilmarshall commented 7 years ago

Very good :-) BTW you can also close issues from within commit messages, using the magic phrase "fixes #N" or "closes #N" where N is the number of the issue. This is nice because it allows you to go back and see exactly what it was you did to fix the problem. You can also use the same phrases in pull request comments - when the PR is merged, that issue will get closed.