Matthew R. Becker

Results 684 comments of Matthew R. Becker

This is already done here: https://github.com/LSSTDESC/seacliff/blob/main/seacliff/rubin_psf.py

As I said before, seacliff is potentially a very natural home for SSI. imSim can pull in seacliff modules as well.

I have also done the WCS: https://github.com/LSSTDESC/seacliff/blob/main/seacliff/rubin_sky_wcs.py and the noise: https://github.com/LSSTDESC/seacliff/blob/main/seacliff/rubin_noise.py

Ah I missed that! LGTM!

I like all of the flake8 rules so I like preview mode, but that is me.

Let's wait for a few more reviews and then merge.

We have two approvals from core , so I'd be happy to merge today.

We have two test failures. After those are fixed, we can merge.

I don't care personally. I was trying to make sure more discussion was had since others had objected.

I don't care how it is phrased so happy with whatever in terms of allow vs reject lists. One vote in favor of an expanded reject list is that if...