Details
-
Type:
Story
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: afw, astro_metadata_translator, obs_lsst, ts_aos
-
Story Points:2
-
Team:Telescope and Site
-
Urgent?:No
Description
FOCUSZ is not currently in ExposureInfo. We want to add this in so we can access it with the images along with the unit information. As I understand it this requires two steps:
1) Add the focus_z information into ObservationInfo in astro_metadata_translator.
2) Then it can be added to ExposureInfo.
Questions I have:
Should it go into VisitInfo?
How can you add distance units to values in the C++? I see you can use the lsst.geom units for angular things but not sure about linear distance.
Attachments
Activity
Bryce Kalmbach, are you ok with me taking over the afw portion of the review?
Krzysztof Findeisen
added a comment - Bryce Kalmbach , are you ok with me taking over the afw portion of the review? Yeah, of course. Thanks for taking it on!
No objections to the afw part.
Krzysztof Findeisen
added a comment - No objections to the afw part. obs_base changes seem fine.
Requesting reviews on PRs in:
afw: https://github.com/lsst/afw/pull/646
obs_base: https://github.com/lsst/obs_base/pull/425
Also, please let me know if there's anywhere else I still need to make changes. Thanks!