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

ChebyshevBoundedField should use _ not . as field separators for persistence

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None

      Description

      ChebyshevBoundedField uses "." instead of "_" as field separators in its afw table persistence. This is the old way of doing things, and unfortunately causes errors when reading in older versions of tables, becaus afw converts "." to "_" in that situation.

      This shows up as a unit test failure in DM-2981 (brought over from HSC) when an older version table is read in.

      It is an open question whether to fix this as part of DM-2981 (which conveniently has a test that shows the problem, though not intentionally so) or separately, in which case a new test is wanted. In the former case I'm happy to do the work so I can finish DM-2981.

      Many thanks to Jim Bosch for diagnosing the problem.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-2981 polygon masking in CoaddPsf

          • Done
          is triggered by

          RFC - An issue whose primary purpose is to advertise an intent and allow stakeholders to comment. RFC-58 Refactor afw.math.Approximate, .BackgroundMI, .GaussianProcess to share interface

          • Implemented

            Activity

            Ascending order - Click to sort in descending order
            rowen Russell Owen created issue -
            rowen Russell Owen made changes -
            Field Original Value New Value
            Link This issue blocks DM-2981 [ DM-2981 ]
            swinbank John Swinbank made changes -
            Description ChebyshevBoundedField uses "." instead of "_" as field separators in its afw table persistence. This is the old way of doing things, and unfortunately causes errors when reading in older versions of tables, becaus afw converts "." to "_" in that situation.

            This shows up as a unit test failure in DM-2981 (brought over from HSC) when an older version table is read in.

            It is an open question whether to fix this as part of DM-2981 (which conveniently has a test that shows the problem, though not intentionally so) or separately, in which case a new test is wanted. In the former case I'm happy to do the work so I can finish DM-2981.             		ChebyshevBoundedField uses "." instead of "\_" as field separators in its afw table persistence. This is the old way of doing things, and unfortunately causes errors when reading in older versions of tables, becaus afw converts "." to "_" in that situation.

            This shows up as a unit test failure in DM-2981 (brought over from HSC) when an older version table is read in.

            It is an open question whether to fix this as part of DM-2981 (which conveniently has a test that shows the problem, though not intentionally so) or separately, in which case a new test is wanted. In the former case I'm happy to do the work so I can finish DM-2981.
            rowen Russell Owen made changes -
            Description ChebyshevBoundedField uses "." instead of "\_" as field separators in its afw table persistence. This is the old way of doing things, and unfortunately causes errors when reading in older versions of tables, becaus afw converts "." to "_" in that situation.

            This shows up as a unit test failure in DM-2981 (brought over from HSC) when an older version table is read in.

            It is an open question whether to fix this as part of DM-2981 (which conveniently has a test that shows the problem, though not intentionally so) or separately, in which case a new test is wanted. In the former case I'm happy to do the work so I can finish DM-2981.             		ChebyshevBoundedField uses "." instead of "\_" as field separators in its afw table persistence. This is the old way of doing things, and unfortunately causes errors when reading in older versions of tables, becaus afw converts "." to "_" in that situation.

            This shows up as a unit test failure in DM-2981 (brought over from HSC) when an older version table is read in.

            It is an open question whether to fix this as part of DM-2981 (which conveniently has a test that shows the problem, though not intentionally so) or separately, in which case a new test is wanted. In the former case I'm happy to do the work so I can finish DM-2981.

            Many thanks to Jim Bosch for diagnosing the problem.
            rowen Russell Owen made changes -
            Assignee Russell Owen [ rowen ]
            rowen Russell Owen made changes -
            Status To Do [ 10001 ] In Progress [ 3 ]
            rowen Russell Owen made changes -
            Sprint Science Pipelines DM-S15-5 [ 162 ]
            Story Points 1
            Team Alert Production [ 10300 ]
            rowen Russell Owen made changes -
            Resolution Done [ 10000 ]
            Status In Progress [ 3 ] Done [ 10002 ]
            swinbank John Swinbank made changes -
            Epic Link DM-1991 [ 16080 ]
            tjenness Tim Jenness made changes -
            Link This issue is triggered by RFC-58 [ RFC-58 ]

              People

              Assignee:
              rowen Russell Owen
              Reporter:
              rowen Russell Owen
              Watchers:
              Jim Bosch, John Swinbank, Russell Owen, Simon Krughoff, Yusra AlSayyad
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.