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

Port meas_extensions_photometryKron to Python 3

    XMLWordPrintable

    Details

      Attachments

        Issue Links

          Activity

          Hide
          fred3m Fred Moolekamp added a comment -

          Python 3 porting complete. Python 2 and 3 builds on Jenkins passed.

          Show
          fred3m Fred Moolekamp added a comment - Python 3 porting complete. Python 2 and 3 builds on Jenkins passed.
          Hide
          tjenness Tim Jenness added a comment -

          The python 3 bits look okay to me. I tried not to get distracted by the weird test that allows you to skip the test failures and the bit of the test that catches the test failure so it can print a bit of extra information.

          Show
          tjenness Tim Jenness added a comment - The python 3 bits look okay to me. I tried not to get distracted by the weird test that allows you to skip the test failures and the bit of the test that catches the test failure so it can print a bit of extra information.
          Hide
          fred3m Fred Moolekamp added a comment -

          That is strange. I'm not sure what good a test is if it doesn't actually fail when it is supposed to.

          Show
          fred3m Fred Moolekamp added a comment - That is strange. I'm not sure what good a test is if it doesn't actually fail when it is supposed to.
          Hide
          tjenness Tim Jenness added a comment -

          Yes, trying not to go there but it must have been for a reason because Jim Bosch seems to have been the person that coded that up. I would have preferred a sentence of justification but off topic for this ticket.

          Show
          tjenness Tim Jenness added a comment - Yes, trying not to go there but it must have been for a reason because Jim Bosch seems to have been the person that coded that up. I would have preferred a sentence of justification but off topic for this ticket.

            People

            Assignee:
            fred3m Fred Moolekamp
            Reporter:
            tjenness Tim Jenness
            Reviewers:
            Tim Jenness
            Watchers:
            Fred Moolekamp, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Dates

              Created:
              Updated:
              Resolved: