# Remove obsolete casts and vectors from Python

XMLWordPrintable

#### Details

• Type: Story
• Status: Done
• Resolution: Done
• Fix Version/s: None
• Component/s:
• Labels:
• Story Points:
1
• Sprint:
• Team:

#### Activity

Hide
Russell Owen added a comment - - edited

In most cases this updates code that not unit tested, including examples, display code and a bit of other library code.

In obs_sdss I fixed a problem that prevented lsst_dm_stack_demo from running under python 2 (which I would have caught as part of DM-9429 if Jenkins could have been used to run the stack demo at that point, as it now can).

Show
Russell Owen added a comment - - edited In most cases this updates code that not unit tested, including examples, display code and a bit of other library code. In obs_sdss I fixed a problem that prevented lsst_dm_stack_demo from running under python 2 (which I would have caught as part of DM-9429 if Jenkins could have been used to run the stack demo at that point, as it now can).
Hide
Russell Owen added a comment -

I encourage the reviewer to start with obs_sdss (so the demo can run) and afw (because the changes are innocuous and so many packages rely on afw).

Show
Russell Owen added a comment - I encourage the reviewer to start with obs_sdss (so the demo can run) and afw (because the changes are innocuous and so many packages rely on afw).
Hide
Krzysztof Findeisen added a comment -

All changes approved. Some minor comments on individual packages' pull requests.

Show
Krzysztof Findeisen added a comment - All changes approved. Some minor comments on individual packages' pull requests.
Hide
Krzysztof Findeisen added a comment -

One meta-comment: please try to follow stack conventions on PR names in the future, https://developer.lsst.io/processes/workflow.html#make-a-pull-request. The PR names for this issue were quite diverse.

Show
Krzysztof Findeisen added a comment - One meta-comment: please try to follow stack conventions on PR names in the future, https://developer.lsst.io/processes/workflow.html#make-a-pull-request . The PR names for this issue were quite diverse.
Hide
Russell Owen added a comment -

Thank you for the quick and helpful review. All changes merged (after rebasing one set of changes to clarify why the names of files in examples were changed).

Show
Russell Owen added a comment - Thank you for the quick and helpful review. All changes merged (after rebasing one set of changes to clarify why the names of files in examples were changed).
Hide
Pim Schellart [X] (Inactive) added a comment -

I'm guilty of that as well. I forgot we had conventions for these and always just used whatever github came up with.

Show
Pim Schellart [X] (Inactive) added a comment - I'm guilty of that as well. I forgot we had conventions for these and always just used whatever github came up with.
Hide
Russell Owen added a comment -

I added DM-9436 to handle making afw's spatialCellExample.py run again. It is lower priority and needs more work than I felt was justified in this ticket.

At this point we can build all of lsst_distrib in Jenkins and the stack demo will run. Note that meas_multifit is not yet wrapped, so the build only works due to a hack in obs_subaru to not pull in that package. Also there are temporary hack to disable ctrl_events and ctrl_orca, both of which will be retired very shortly and so were not worth converting to pybind11.

Show
Russell Owen added a comment - I had no idea we even had conventions for PR names. I'll read the link. I added DM-9436 to handle making afw's spatialCellExample.py run again. It is lower priority and needs more work than I felt was justified in this ticket. At this point we can build all of lsst_distrib in Jenkins and the stack demo will run. Note that meas_multifit is not yet wrapped, so the build only works due to a hack in obs_subaru to not pull in that package. Also there are temporary hack to disable ctrl_events and ctrl_orca , both of which will be retired very shortly and so were not worth converting to pybind11.

#### People

Assignee:
Russell Owen
Reporter:
Russell Owen
Reviewers:
Krzysztof Findeisen
Watchers:
Krzysztof Findeisen, Pim Schellart [X] (Inactive), Russell Owen