Chris Walter

Great. We should have you come and talk about seacliff in a meeting (I'm still not exactly sure what it is) and also help us understand how we could use...

Note from Jim in slack about the current situation with QE throughputs. > For the baseline set, I see only one file called detector.dat that has a throughput curve. The...

From another closed issue note: "Currently there is a separate BF pixel distortion file for ITL and E2V, and one overall scaling strength parameter in imSim"

I think (hope!) we finally removed the old PhoSim coordinate convention. But, I thought we had some switches in the YAML to flip if really necessary. I don't see that...

@jchiang87 @rmjarvis I see a flip_g2 variable in the instcat code, but I don't see where to set it (I think this is left around from before actually). I don't...

Hi All, Pinging people again to look at this issue and see if it should be closed now.

Pinging people again on this. So I can see that the parameter flip_g2 is set to True by default here: https://github.com/LSSTDESC/imSim/blob/18b03e764cf396fbc1ecc7f35d3eed207a921c5e/imsim/instcat.py#L104 and is used to set the sign later. Is...

I thought it gets set to True here: ``` def __init__(self, file_name, wcs, xsize=4096, ysize=4096, sed_dir=None, edge_pix=100, sort_mag=True, flip_g2=True, min_source=None, skip_invalid=True, logger=None): ``` which is why I was confused. Is...

So, I think I would like to not flip it for people writing new files and let people know if they want to use a previous DC2 catalog they should...

Well, I want to erase all remnants of this convention as a default from our code. I objected strongly at the time we decided to keep this because changing it...