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

Port meas_modelfit to Python 3

    XMLWordPrintable

Details

    Attachments

      Issue Links

        Activity

          No builds found.
          tjenness Tim Jenness created issue -
          tjenness Tim Jenness made changes -
          Field Original Value New Value
          Link This issue relates to DM-6179 [ DM-6179 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          nlust Nate Lust made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Link This issue is blocked by DM-7289 [ DM-7289 ]
          tjenness Tim Jenness made changes -
          Link This issue is blocked by DM-7292 [ DM-7292 ]
          tjenness Tim Jenness made changes -
          Link This issue is blocked by DM-7256 [ DM-7256 ]
          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-7262 [ DM-7262 ]
          nlust Nate Lust made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Link This issue is blocked by DM-7358 [ DM-7358 ]
          swinbank John Swinbank made changes -
          Epic Link DM-7318 [ 26403 ]
          swinbank John Swinbank made changes -
          Team Data Release Production [ 10301 ]
          swinbank John Swinbank made changes -
          Story Points 2
          swinbank John Swinbank made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          swinbank John Swinbank made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          fred3m Fred Moolekamp made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          fred3m Fred Moolekamp made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          swinbank John Swinbank made changes -
          Assignee Fred Moolekamp [ fred3m ]
          swinbank John Swinbank made changes -
          Sprint DRP F16-4 [ 246 ]
          fred3m Fred Moolekamp made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          fred3m Fred Moolekamp made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          fred3m Fred Moolekamp made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          Parejkoj John Parejko made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          Parejkoj John Parejko made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          spietrowicz Steve Pietrowicz made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          rowen Russell Owen made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          rowen Russell Owen made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          swinbank John Swinbank made changes -
          Sprint DRP F16-4 [ 246 ] DRP F16-5 [ 252 ]
          swinbank John Swinbank made changes -
          Sprint DRP F16-5 [ 252 ]
          swinbank John Swinbank made changes -
          Epic Link DM-7318 [ 26403 ] DM-8136 [ 27591 ]
          Parejkoj John Parejko made changes -
          Sprint Alert Production F16 - 11c [ 296 ]
          Parejkoj John Parejko made changes -
          Assignee Fred Moolekamp [ fred3m ] John Parejko [ parejkoj ]
          Parejkoj John Parejko added a comment -

          Doing this, so we can be done with the python3 port!

          Parejkoj John Parejko added a comment - Doing this, so we can be done with the python3 port!
          Parejkoj John Parejko made changes -
          Status To Do [ 10001 ] In Progress [ 3 ]
          Parejkoj John Parejko added a comment -

          Fred: can you please review this quickly? Note that there is a SWIG addition in afw, which may not be ideal, but should be enough to make things work for now. We'll want a pybind11 version of those methods, and some unittests to go with them eventually.

          Once this is done, the last step is obs_subaru, which I've already ported but it failed when running a modelfit test...

          Parejkoj John Parejko added a comment - Fred: can you please review this quickly? Note that there is a SWIG addition in afw, which may not be ideal, but should be enough to make things work for now. We'll want a pybind11 version of those methods, and some unittests to go with them eventually. Once this is done, the last step is obs_subaru, which I've already ported but it failed when running a modelfit test...
          Parejkoj John Parejko made changes -
          Status In Progress [ 3 ] In Review [ 10004 ]
          Reviewers Fred Moolekamp [ fred3m ]

          Looks good to me. I noticed that you removed a couple of shebangs, is that standard procedure now, or is that something that was done automatically?

          You should probably open a ticket to cover adding tests for the setting strings in afw table. I'm wrapping that in pybind11 right now and noticed that there are no tests available for setting strings, but I think that the code should be much easier in pybind11 than SWIG and once a test is in place the pybind11 stack can be modified.

          fred3m Fred Moolekamp added a comment - Looks good to me. I noticed that you removed a couple of shebangs, is that standard procedure now, or is that something that was done automatically? You should probably open a ticket to cover adding tests for the setting strings in afw table. I'm wrapping that in pybind11 right now and noticed that there are no tests available for setting strings, but I think that the code should be much easier in pybind11 than SWIG and once a test is in place the pybind11 stack can be modified.
          fred3m Fred Moolekamp made changes -
          Status In Review [ 10004 ] Reviewed [ 10101 ]
          Parejkoj John Parejko made changes -
          Link This issue relates to DM-8243 [ DM-8243 ]

          The shebangs I removed were from code not in bin/, so I don't see how they do any good anyway: that code can't really be executed as it's designed for import. I'm not sure that there is any policy on it either way.

          I added DM-8243 and put you on the watch list, Fred.

          Jenkins py2/3 lsst_py3 pass for the record, after rebasing afw: https://ci.lsst.codes/job/stack-os-matrix/18099/

          Parejkoj John Parejko added a comment - The shebangs I removed were from code not in bin/, so I don't see how they do any good anyway: that code can't really be executed as it's designed for import. I'm not sure that there is any policy on it either way. I added DM-8243 and put you on the watch list, Fred. Jenkins py2/3 lsst_py3 pass for the record, after rebasing afw: https://ci.lsst.codes/job/stack-os-matrix/18099/
          Parejkoj John Parejko added a comment -

          Thanks for such a fast review, Fred. Merged and done.

          Parejkoj John Parejko added a comment - Thanks for such a fast review, Fred. Merged and done.
          Parejkoj John Parejko made changes -
          Resolution Done [ 10000 ]
          Status Reviewed [ 10101 ] Done [ 10002 ]
          Parejkoj John Parejko made changes -
          Team Data Release Production [ 10301 ] Alert Production [ 10300 ]
          tjenness Tim Jenness made changes -
          Remote Link This issue links to "Page (Confluence)" [ 14233 ] This issue links to "Page (Confluence)" [ 14233 ]
          swinbank John Swinbank made changes -
          Epic Link DM-8136 [ 27591 ]
          Parejkoj John Parejko made changes -
          Link This issue relates to DM-8528 [ DM-8528 ]
          krughoff Simon Krughoff (Inactive) made changes -
          Epic Link DM-7362 [ 26448 ]

          People

            Parejkoj John Parejko
            tjenness Tim Jenness
            Fred Moolekamp
            Fred Moolekamp, John Parejko, Tim Jenness
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Jenkins

                No builds found.