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

Get Augustin's reduction pipeline running

    XMLWordPrintable

    Details

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

      Description

      git clone https://github.com/guyonnet/slitless_image_reduction/

      clone croaks & install it

      Install and configure SExtractor

      Get the pipeline running.

      Then port it to py3 so it can be run from the same env as the stack.

       

        Attachments

          Issue Links

            Activity

            No builds found.
            mfisherlevine Merlin Fisher-Levine created issue -
            Show
            mfisherlevine Merlin Fisher-Levine added a comment - The py3 (and PEP8) update of https://github.com/guyonnet/slitless_image_reduction/ can be found here: https://github.com/lsst-dm/slitless_image_reduction/tree/tickets/DM-16699
            mfisherlevine Merlin Fisher-Levine made changes -
            Field Original Value New Value
            Link This issue relates to DM-16699 [ DM-16699 ]
            mfisherlevine Merlin Fisher-Levine made changes -
            Status To Do [ 10001 ] In Review [ 10004 ]
            Hide
            swinbank John Swinbank added a comment -

            There's a typo on line 145 of telinst.py. Surprised you missed it.

            ...

            I jest, of course. More seriously, I do wonder why this “py3” update contains a bunch of files in a directory called local/lib/python2.7/site-packages/. However, since this isn't code we expect to comply with LSST standards, if you are happy that it's working then I don't propose to think any harder about that.

            One minor point for added cleanliness: how about running —

            $ find /path/to/package -name \*pyc -delete && git commit -am "Remove .pyc files."
            

            ?

            Show
            swinbank John Swinbank added a comment - There's a typo on line 145 of telinst.py . Surprised you missed it. ... I jest, of course. More seriously, I do wonder why this “py3” update contains a bunch of files in a directory called local/lib/python2.7/site-packages/ . However, since this isn't code we expect to comply with LSST standards, if you are happy that it's working then I don't propose to think any harder about that. One minor point for added cleanliness: how about running — $ find /path/to/package -name \*pyc -delete && git commit -am "Remove .pyc files." ?
            swinbank John Swinbank made changes -
            Status In Review [ 10004 ] Reviewed [ 10101 ]
            Hide
            mfisherlevine Merlin Fisher-Levine added a comment -

            Suggestion to delete *.pyc done, and merged, thanks!

            Show
            mfisherlevine Merlin Fisher-Levine added a comment - Suggestion to delete *.pyc done, and merged, thanks!
            mfisherlevine Merlin Fisher-Levine made changes -
            Resolution Done [ 10000 ]
            Status Reviewed [ 10101 ] Done [ 10002 ]
            mfisherlevine Merlin Fisher-Levine made changes -
            Epic Link DM-12816 [ 36463 ]
            swinbank John Swinbank made changes -
            Team Data Release Production [ 10301 ]

              People

              Assignee:
              mfisherlevine Merlin Fisher-Levine
              Reporter:
              mfisherlevine Merlin Fisher-Levine
              Reviewers:
              John Swinbank
              Watchers:
              John Swinbank, Merlin Fisher-Levine
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.