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

Port daf_persistence to Python 3

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: daf_persistence
    • Labels:
      None

      Description

      Work relating to getting daf_persistence to run on python 3. Includes some code modernization.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Nate Pease can you please review my changes? They are needed before we can progress on the Python 3 port. I have made extensive cleanups to the code base as well and made it so the tests don't have to be run from the daf_persistence directory.

            I have one remaining test failure with butlerAlias.py where keys are being stored as bytes and not strings. I'm not entirely sure where in the code keys and values are stored in the dicts and why they would be bytes (bytes would be if the key comes from an external source). I'm not familiar enough with the code base to know where to look so your help would be much appreciated.

            Show
            tjenness Tim Jenness added a comment - Nate Pease can you please review my changes? They are needed before we can progress on the Python 3 port. I have made extensive cleanups to the code base as well and made it so the tests don't have to be run from the daf_persistence directory. I have one remaining test failure with butlerAlias.py where keys are being stored as bytes and not strings. I'm not entirely sure where in the code keys and values are stored in the dicts and why they would be bytes (bytes would be if the key comes from an external source). I'm not familiar enough with the code base to know where to look so your help would be much appreciated.
            Hide
            tjenness Tim Jenness added a comment -

            Merged.

            Show
            tjenness Tim Jenness added a comment - Merged.

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Nate Pease
              Watchers:
              Hsin-Fang Chiang, Nate Pease, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: