New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Tickets/DM-13325: warpExposure does not propagate visitInfo #311
Conversation
b29902b
to
6bfeeea
Compare
include/lsst/afw/math/warpExposure.h
Outdated
@@ -383,7 +383,7 @@ class WarpingControl { | |||
template <typename DestExposureT, typename SrcExposureT> | |||
int warpExposure( | |||
DestExposureT &destExposure, ///< Remapped exposure. Wcs and xy0 are read, MaskedImage is set, | |||
///< and Calib and Filter are copied from srcExposure. | |||
///< and Calib, Filter, visitInfo, are copied from srcExposure. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest putting "and" before visitInfo
tests/test_warpExposure.py
Outdated
data.rotType, | ||
data.observatory, | ||
data.weather, | ||
) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I suggest you replace all this with the following and remove the earlier import of collections
def makeVisitInfo():
"""Return a non-NaN visitInfo."""
return afwImage.VisitInfo(exposureId=10313423,
exposureTime=10.01,
darkTime=11.02,
date=dafBase.DateTime(65321.1, dafBase.DateTime.MJD, dafBase.DateTime.TAI),
ut1=12345.1,
era=45.1*afwGeom.degrees,
boresightRaDec=afwCoord.IcrsCoord(23.1*afwGeom.degrees, 73.2*afwGeom.degrees),
boresightAzAlt=afwCoord.Coord(134.5*afwGeom.degrees, 33.3*afwGeom.degrees),
boresightAirmass=1.73,
boresightRotAngle=73.2*afwGeom.degrees,
rotType=afwImage.RotType.SKY,
observatory=afwCoord.Observatory(
11.1*afwGeom.degrees, 22.2*afwGeom.degrees, 0.333),
weather=afwCoord.Weather(1.1, 2.2, 34.5),
)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically looks good. I had a few changes I would like to see, but nothing that requires another review.
Modify destExposure note to reflect new code
548d2ab
to
064b153
Compare
No description provided.