Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-7324

Port ci_hsc to Python 3

    Details

      Description

      Work associated with ensuring that ci_hsc works with Python 3.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment - - edited

            ci_hsc can not be ported to Python 3 until scons itself works with Python 3, or this package is rewritten so it does not use scons for pipeline task execution. I've added a blocker ticket to reflect that dependency.

            Show
            tjenness Tim Jenness added a comment - - edited ci_hsc can not be ported to Python 3 until scons itself works with Python 3, or this package is rewritten so it does not use scons for pipeline task execution. I've added a blocker ticket to reflect that dependency.
            Hide
            tjenness Tim Jenness added a comment -

            DM-8560 is far enough along (lsst_distrib now builds) that I think we are in a good position to try ci_hsc. You will currently need to build your stack with tickets/DM-8560 branch so as to pick up SCons3 and the new sconsUtils.

            Show
            tjenness Tim Jenness added a comment - DM-8560 is far enough along (lsst_distrib now builds) that I think we are in a good position to try ci_hsc. You will currently need to build your stack with tickets/ DM-8560 branch so as to pick up SCons3 and the new sconsUtils .
            Hide
            nlust Nate Lust added a comment -

            This has run successfully on both py2 and py3

            Show
            nlust Nate Lust added a comment - This has run successfully on both py2 and py3
            Hide
            tjenness Tim Jenness added a comment -

            This looks fine except that I don't think the iter() wrappers should be there. They aren't needed at all on Python3 and I don't think they help on Python2 because the list still has to be created to turn it into an iterators so you won't see any speed up from using iter().

            Show
            tjenness Tim Jenness added a comment - This looks fine except that I don't think the iter() wrappers should be there. They aren't needed at all on Python3 and I don't think they help on Python2 because the list still has to be created to turn it into an iterators so you won't see any speed up from using iter() .
            Hide
            tjenness Tim Jenness added a comment -

            Thanks for updating the PR. Looks good now.

            Show
            tjenness Tim Jenness added a comment - Thanks for updating the PR. Looks good now.

              People

              • Assignee:
                nlust Nate Lust
                Reporter:
                tjenness Tim Jenness
                Reviewers:
                Tim Jenness
                Watchers:
                Nate Lust, Paul Price, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel