Details
-
Type:
Improvement
-
Status: Done
-
Resolution: Done
-
Fix Version/s: None
-
Component/s: ip_isr, obs_base, obs_cfht, obs_decam, obs_lsstSim, obs_sdss, obs_subaru, obs_test, pipe_drivers, pipe_tasks
-
Labels:
-
Story Points:0.5
-
Epic Link:
-
Sprint:Alert Production S17 - 3
-
Team:Alert Production
Description
lsst.afw.image.makeVisitInfo(...) is superseded by lsst.afw.image.VisitInfo(...). Remove existing usage of makeVisitInfo.
makeVisitInfo is used in one or two places in most obs_ packages and a few other packages as well.
Note that this can be done on a package by package basis as time permits. Once all usage is gone we can get rid of makeVisitInfo.
Attachments
Issue Links
- relates to
-
DM-9853 Clean up makeVisitInfo docstring
- Done
Activity
| Assignee | Russell Owen [ rowen ] |
| Component/s | pipe_tasks [ 10726 ] | |
| Component/s | meas_base, pipe_tasks [ 13414 ] |
| Story Points | 2 | 0.5 |
Do you have time to look at this? It's a lot of packages, but the changes are completely trivial.
Note that I accidentally pushed the change to pipe_drivers to master instead of the ticket branch. I can undo that if you want.
| Reviewers | Paul Price [ price ] | |
| Status | To Do [ 10001 ] | In Review [ 10004 ] |
| Epic Link |
|
| Sprint | Alert Production S17 - 3 [ 605 ] | |
| Team | Alert Production [ 10300 ] |
Trivial changes, no worries.
Don't worry about the accidental push to pipe_drivers master.
Paul Price
added a comment - Trivial changes, no worries.
Don't worry about the accidental push to pipe_drivers master. Paul Price
made changes -
| Status | In Review [ 10004 ] | Reviewed [ 10101 ] |
| Resolution | Done [ 10000 ] | |
| Status | Reviewed [ 10101 ] | Done [ 10002 ] |
I somehow managed to get a ticket branch pushed that contained some incorrect changes. Corrected, but the result is an invalid pull request in addition to the good one.