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

lsst.meas.base.tests.TestDataset allows randomness

    Details

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

      Description

      lsst.meas.base.tests.TestDataset contains two methods (makePerturbedWcs and realize) that take an optional seed for the random number generator. In both cases, the seed defaults to None, which means the RNG is seeded in some non-deterministic manner. However, this is intended for tests, which we require are deterministic. The current API therefore makes it too easy (simply neglecting to specify the optional randomSeed keyword argument) to do something bad. We should default to some deterministic value for the seed.

        Attachments

          Activity

            People

            • Assignee:
              price Paul Price
              Reporter:
              price Paul Price
              Reviewers:
              John Swinbank
              Watchers:
              John Swinbank, Paul Price
            • Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel