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

Implement lossless FITS compression

    Details

    • Type: Story
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: obs_base
    • Labels:
      None
    • Team:
      External

      Description

      Following adoption of RFC-378, make lossless FITS compression the default.

        Attachments

          Issue Links

            Activity

            Hide
            price Paul Price added a comment -

            Kian-Tat Lim, would you sign off on this, please?

            price@pap-laptop:~/LSST/obs_base (tickets/DM-12599=) $ git sub
            commit ce1699f7837b6a33a467b790635d2c29b8d4c045
            Author: Paul Price <price@astro.princeton.edu>
            Date:   Mon Nov 13 13:51:19 2017 -0500
             
                writeRecipes: change how the default is set
                
                Set the default through a symbolic name in the YAML, rather than
                explicitly calling it the "default". This allows the default to be
                changed more simply, and gives a symbolic name to the default besides
                just "default".
             
             policy/writeRecipes.yaml | 13 ++++++++-----
             1 file changed, 8 insertions(+), 5 deletions(-)
             
            commit 8d4e02aa4fd231abb33cf37f38d011073e78180d (HEAD -> tickets/DM-12599, origin/tickets/DM-12599)
            Author: Paul Price <price@astro.princeton.edu>
            Date:   Mon Nov 13 13:52:22 2017 -0500
             
                writeRecipes: set default to lossless compression
                
                Following adoption of RFC-378, lossless compression is to be the
                default.
             
             policy/writeRecipes.yaml | 4 ++--
             1 file changed, 2 insertions(+), 2 deletions(-)
            

            Show
            price Paul Price added a comment - Kian-Tat Lim , would you sign off on this, please? price@pap-laptop:~/LSST/obs_base (tickets/DM-12599=) $ git sub commit ce1699f7837b6a33a467b790635d2c29b8d4c045 Author: Paul Price <price@astro.princeton.edu> Date: Mon Nov 13 13:51:19 2017 -0500   writeRecipes: change how the default is set Set the default through a symbolic name in the YAML, rather than explicitly calling it the "default". This allows the default to be changed more simply, and gives a symbolic name to the default besides just "default".   policy/writeRecipes.yaml | 13 ++++++++----- 1 file changed, 8 insertions(+), 5 deletions(-)   commit 8d4e02aa4fd231abb33cf37f38d011073e78180d (HEAD -> tickets/DM-12599, origin/tickets/DM-12599) Author: Paul Price <price@astro.princeton.edu> Date: Mon Nov 13 13:52:22 2017 -0500   writeRecipes: set default to lossless compression Following adoption of RFC-378, lossless compression is to be the default.   policy/writeRecipes.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)
            Hide
            ktl Kian-Tat Lim added a comment -

            LGTM.

            Show
            ktl Kian-Tat Lim added a comment - LGTM.
            Hide
            price Paul Price added a comment -

            Thanks KT. Took your suggestion of doing it slightly differently.
            The original passed Jenkins. The modified version has gone through ci_hsc independently.

            Show
            price Paul Price added a comment - Thanks KT. Took your suggestion of doing it slightly differently. The original passed Jenkins . The modified version has gone through ci_hsc independently.

              People

              • Assignee:
                price Paul Price
                Reporter:
                price Paul Price
                Reviewers:
                Kian-Tat Lim
                Watchers:
                Gregory Dubois-Felsmann, Kian-Tat Lim, Paul Price
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: