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

Replace usage of execfile in astrometryNetDataConfig.py

    XMLWordPrintable

    Details

      Description

      astrometryNetDataConfig.py gained a from past.builtins import execfile during the python3 conversion. We should replace this with a more modern syntax.

        Attachments

          Issue Links

            Activity

            Show
            tjenness Tim Jenness added a comment - See http://stackoverflow.com/questions/436198/what-is-an-alternative-to-execfile-in-python-3-0#437857
            Hide
            Parejkoj John Parejko added a comment -

            Tim Jenness You pointed me to the solution, do you mind a very short review?

            The jointcal warnings DM-12925 went away once I implemented this.

            Show
            Parejkoj John Parejko added a comment - Tim Jenness You pointed me to the solution, do you mind a very short review? The jointcal warnings DM-12925 went away once I implemented this.
            Hide
            Parejkoj John Parejko added a comment -

            Thanks for the speedy review.

            Merged and done. May your pytest lots be less polluted.

            Show
            Parejkoj John Parejko added a comment - Thanks for the speedy review. Merged and done. May your pytest lots be less polluted.

              People

              Assignee:
              Parejkoj John Parejko
              Reporter:
              Parejkoj John Parejko
              Reviewers:
              Tim Jenness
              Watchers:
              John Parejko, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.