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

Check lmfit works with Python 3

    XMLWordPrintable

    Details

      Description

      lmfit is meant to work with Python 3 but this needs to be checked. Some code in the tree does not work with Python 3 but it is possible that this code is not used by LSST.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            The PyPI page indicates that lmfit works on python 3.5 (at least v0.9.5 does, we currently use 0.9.3).

            Show
            tjenness Tim Jenness added a comment - The PyPI page indicates that lmfit works on python 3.5 (at least v0.9.5 does, we currently use 0.9.3).
            Hide
            Parejkoj John Parejko added a comment -

            Assigning to Meredith, since she'll check on this while doing ip_diffim. If we do have to pull down a new external package, I'll take care of that.

            Show
            Parejkoj John Parejko added a comment - Assigning to Meredith, since she'll check on this while doing ip_diffim. If we do have to pull down a new external package, I'll take care of that.
            Hide
            tjenness Tim Jenness added a comment -

            This clearly worked so I think this ticket should be closed. There was no need to update this package.

            Show
            tjenness Tim Jenness added a comment - This clearly worked so I think this ticket should be closed. There was no need to update this package.
            Hide
            mrawls Meredith Rawls added a comment -

            OK. Should I mark it as "done" or some other status since it wasn't actually an issue?

            Show
            mrawls Meredith Rawls added a comment - OK. Should I mark it as "done" or some other status since it wasn't actually an issue?
            Hide
            tjenness Tim Jenness added a comment -

            The phrasing was "check" rather than "update" so I'd go with "Done" but assign 0 SP.

            Show
            tjenness Tim Jenness added a comment - The phrasing was "check" rather than "update" so I'd go with "Done" but assign 0 SP.
            Hide
            mrawls Meredith Rawls added a comment -

            As discussed in the comments, since ip_diffim now works with py3, lmfit should not need updating.

            Show
            mrawls Meredith Rawls added a comment - As discussed in the comments, since ip_diffim now works with py3, lmfit should not need updating.

              People

              Assignee:
              mrawls Meredith Rawls
              Reporter:
              tjenness Tim Jenness
              Watchers:
              John Parejko, Meredith Rawls, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.