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

Test and fix repr implementations for Box, Point, and Exten

    XMLWordPrintable

    Details

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

      Description

      While working on a repr implementation that delegates to those in geom, I discovered that they're broken in various small ways that breaks eval roundtrip.  Since these are classes that could support eval(repr(...)) roundtrip, they should.

      Fix and new tests are ready; just need a branch to put it on.

        Attachments

          Issue Links

          is duplicated by

          Bug - A problem which impairs or prevents the functions of the product. DM-16740 Box2I stringification no longer matches ctor

          • Invalid

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jbosch Jim Bosch added a comment -

            Tiny fix for a tiny problem (and while there may be other similar problems in this package, I'm not interested in a scope increase right now).  John Parejko , could you review?

            PR is https://github.com/lsst/geom/pull/37

             

            Show
            jbosch Jim Bosch added a comment - Tiny fix for a tiny problem (and while there may be other similar problems in this package, I'm not interested in a scope increase right now).  John Parejko , could you review? PR is https://github.com/lsst/geom/pull/37  
            Hide
            Permalink
            Parejkoj John Parejko added a comment -

            A few comments on the PR. Thanks for doing this!

            Show
            Parejkoj John Parejko added a comment - A few comments on the PR. Thanks for doing this!

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Reviewers:
              John Parejko
              Watchers:
              Jim Bosch, John Parejko
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.