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

Port daf_ingest to Python 3

    Details

    • Type: Story
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_ingest
    • Labels:
      None
    • Team:
      Data Access and Database

      Description

      daf_ingest requires some minor changes to get it working on Python 3.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Serge Monkewitz sorry. You are still the default assignee for daf_ingest.

            Show
            tjenness Tim Jenness added a comment - Serge Monkewitz sorry. You are still the default assignee for daf_ingest .
            Hide
            tjenness Tim Jenness added a comment -

            Please pay particular attention to this comment when porting: https://github.com/lsst/daf_ingest/blob/master/python/lsst/daf/ingest/indexExposure.py#L177

            Show
            tjenness Tim Jenness added a comment - Please pay particular attention to this comment when porting: https://github.com/lsst/daf_ingest/blob/master/python/lsst/daf/ingest/indexExposure.py#L177
            Hide
            tjenness Tim Jenness added a comment -

            I've had a quick look at the Python 3 situation and there are a couple of issues that need some thought. The first is the buffer problem mentioned earlier. The second is the loose way that bytes and strings are handled so there is some uncertainty over which parts of ingest should really be bytes and where they should be strings and whether the place where that conversion happens is in the right place. I've put some initial patches on this ticket branch.

            Show
            tjenness Tim Jenness added a comment - I've had a quick look at the Python 3 situation and there are a couple of issues that need some thought. The first is the buffer problem mentioned earlier. The second is the loose way that bytes and strings are handled so there is some uncertainty over which parts of ingest should really be bytes and where they should be strings and whether the place where that conversion happens is in the right place. I've put some initial patches on this ticket branch.
            Hide
            tjenness Tim Jenness added a comment -

            daf_ingest is not a supported package. Closing this as won't fix.

            Show
            tjenness Tim Jenness added a comment - daf_ingest is not a supported package. Closing this as won't fix.

              People

              • Assignee:
                Unassigned
                Reporter:
                tjenness Tim Jenness
                Watchers:
                Fritz Mueller, Tim Jenness
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Summary Panel