This pull request should be merged when obs_lsstSim (or the lack thereof) causes lsst_sims to break.
Do not merge it beforehand, as it will delete (as opposed to replacing) functionality that currently depends on obs_lsstSim.
Jim, Chris: I cannot promise that the orientation of pixel x,y coordinates in sims_GalSimInterface will be correct after this gets merged. I think the obs_lsst PhosimMapper() actually respects the camera team coordinate system, the same way PhoSim does. All this branch does is make sure that tests pass and the code builds. Once obs_lsstSim is gone, ImSim should stop using the camera model in sims_GalSimInterface.
@jchiang87 @cwwalter @jdswinbank
This pull request should be merged when obs_lsstSim (or the lack thereof) causes lsst_sims to break.
Do not merge it beforehand, as it will delete (as opposed to replacing) functionality that currently depends on obs_lsstSim.
Jim, Chris: I cannot promise that the orientation of pixel x,y coordinates in sims_GalSimInterface will be correct after this gets merged. I think the obs_lsst PhosimMapper() actually respects the camera team coordinate system, the same way PhoSim does. All this branch does is make sure that tests pass and the code builds. Once obs_lsstSim is gone, ImSim should stop using the camera model in sims_GalSimInterface.