Uploaded image for project: 'Request For Comments'
  1. Request For Comments
  2. RFC-626

Switch some C++ APIs that use pex_config information from Policy to PropertySet

    Details

    • Type: RFC
    • Status: Implemented
    • Resolution: Done
    • Component/s: DM
    • Labels:
      None

      Description

      We are trying to remove usage of pex_policy in our code so that we can retire it. There are some places that use Policy to receive configuration information (via pexConfig.makePolicy) that I would like to change to use pexConfig.makePropertySet.

      The routines in question are:

      • meas_algorithms.findCosmicRays (called from pipe_tasks and meas_algorithms)
      • ip_diffim (psf matching, kernels and regularization), seemingly only calling makePolicy internally.

      The proposal is:

      1. Change the C++ to use PropertySet
      2. Update the stack code to use the new interface.
      3. Provide a wrapper Python function that looks for the old API usage and warns, before converting the Policy to PropertySet and proceeding.
      4. Formally deprecate pexConfig.makePolicy and the Policy usage for these APIs.

        Attachments

          Issue Links

            Activity

              People

              • Assignee:
                tjenness Tim Jenness
                Reporter:
                tjenness Tim Jenness
                Watchers:
                Colin Slater, John Parejko, John Swinbank, Kian-Tat Lim, Krzysztof Findeisen, Leanne Guy, Michelle Butler, Tim Jenness, Wil O'Mullane
              • Votes:
                0 Vote for this issue
                Watchers:
                9 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:
                  Planned End:

                  Summary Panel