Matthew R. Becker
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...