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

            No builds found.
            tjenness Tim Jenness created issue -
            tjenness Tim Jenness made changes -
            Field Original Value New Value
            Remote Link This issue links to "Page (Confluence)" [ 14369 ]
            smonkewitz Serge Monkewitz made changes -
            Assignee Serge Monkewitz [ smonkewitz ] Tim Jenness [ tjenness ]
            tjenness Tim Jenness made changes -
            Assignee Tim Jenness [ tjenness ]
            tjenness Tim Jenness made changes -
            Team Data Access and Database [ 10204 ]
            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 .
            tjenness Tim Jenness made changes -
            Watchers Tim Jenness [ Tim Jenness ] Fritz Mueller, Tim Jenness [ Fritz Mueller, Tim Jenness ]
            tjenness Tim Jenness made changes -
            Link This issue relates to DM-6179 [ DM-6179 ]
            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
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-7069 [ DM-7069 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-7047 [ DM-7047 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-7245 [ DM-7245 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-7311 [ DM-7311 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-7152 [ DM-7152 ]
            tjenness Tim Jenness made changes -
            Link This issue is blocked by DM-7066 [ DM-7066 ]
            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.
            tjenness Tim Jenness made changes -
            Link This issue relates to RFC-275 [ RFC-275 ]
            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.
            tjenness Tim Jenness made changes -
            Resolution Done [ 10000 ]
            Status To Do [ 10001 ] Won't Fix [ 10405 ]

              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:

                  Jenkins

                  No builds found.