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

Allow translate_header to dump the fixed header

    Details

    • Story Points:
      1
    • Team:
      Architecture

      Description

      Currently translate_header has a dumphdr option to list the file header. This is the actual header and there is no option to provide a fixed up header after astro_metadata_translator has patched things up.

      Add a new option to allow the fixed header to be seen. It's possible that we could do this with the mode switch and make mode=fixed dump the fixed header and dumphdr could alias to mode=header.

        Attachments

          Activity

          Hide
          tjenness Tim Jenness added a comment -

          Merlin Fisher-Levine I added new output modes. My main question for you is the names. Currently I have yaml to dump the header in yaml, "fixed" to dump it in yaml but with the headers fixed up, and "yamlcompact" and "fixedcompact" to dump them in key: value form (by effectively converting the header to a normal dict first). It all works but I'm not happy with the names.

          Show
          tjenness Tim Jenness added a comment - Merlin Fisher-Levine I added new output modes. My main question for you is the names. Currently I have yaml to dump the header in yaml, "fixed" to dump it in yaml but with the headers fixed up, and "yamlcompact" and "fixedcompact" to dump them in key: value form (by effectively converting the header to a normal dict first). It all works but I'm not happy with the names.
          Hide
          tjenness Tim Jenness added a comment -

          Changed my mind a little. yaml and fixed now dump the simple YAML form. yamlnative and fixednative dump the PropertyList/astropy object form.

          Show
          tjenness Tim Jenness added a comment - Changed my mind a little. yaml and fixed now dump the simple YAML form. yamlnative and fixednative dump the PropertyList/astropy object form.
          Hide
          tjenness Tim Jenness added a comment -

          Simon Krughoff thanks for agreeing to do the review.

          Show
          tjenness Tim Jenness added a comment - Simon Krughoff thanks for agreeing to do the review.
          Hide
          krughoff Simon Krughoff added a comment -

          See minor comments in PR.

          Show
          krughoff Simon Krughoff added a comment - See minor comments in PR.

            People

            • Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Simon Krughoff
              Watchers:
              Merlin Fisher-Levine, Simon Krughoff, Tim Jenness
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel