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

Add text file serialization to meas_algorithms Defects class

    XMLWordPrintable

    Details

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

      Description

      It was noted on RFC-595 that the creation of the Defects text files to be stored in the data repositories is distinct from Defects. It would be more useful for people creating defects lists if the Defects class could serialize to the text format.

      Robert Lupton also requests that metadata be stored in the text files (at least the format version number and the detector name and instrument). To satisfy that requirement I will see if I can write out in the Astropy ECSV format.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Simon Krughoff I can now write to and from ecsv format. I include a schema version in the file.

            Show
            tjenness Tim Jenness added a comment - Simon Krughoff I can now write to and from ecsv format. I include a schema version in the file.
            Hide
            krughoff Simon Krughoff added a comment -

            Looks great.

            Show
            krughoff Simon Krughoff added a comment - Looks great.
            Hide
            tjenness Tim Jenness added a comment -

            As part of this I found a minor bug in Astropy and reported it here: https://github.com/astropy/astropy/issues/8673

            Show
            tjenness Tim Jenness added a comment - As part of this I found a minor bug in Astropy and reported it here: https://github.com/astropy/astropy/issues/8673

              People

              Assignee:
              tjenness Tim Jenness
              Reporter:
              tjenness Tim Jenness
              Reviewers:
              Simon Krughoff
              Watchers:
              Robert Lupton, Simon Krughoff, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: