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

Port daf_ingest to Python 3

    XMLWordPrintable

    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 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:

                  CI Builds

                  No builds found.