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

Design better (and Gen3-friendly) way of representing bandpass filters in code

    XMLWordPrintable

    Details

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

      Description

      This is a non-traditional brainstorming RFC to gather preliminary input prior to design work.  The (mostly-)completed design will be RFC'd separately in the future (and I'm not even planning to start that work for several weeks).

      My goal is to replace afw::image::Filter with something that:

      • minimizes the use of singletons;
      • maps to the Gen3 PhysicalFilter and AbstractFilter concepts (probably with separate classes for these);
      • has a sensible relationship with cameraGeom (probably just "Camera has a set of PhysicalFilters");
      • has a sensible relationship with TransmissionCurve (not obvious; could be "PhysicalFilter has TransmissionCurve" or "PhysicalFilter can be used to retrieve a TransmissionCurve from a calibration repo");
      • can be easily mangled into deterministic integer IDs without addition or subtraction of filters breaking old IDs;
      • natively supports or can be extended to support sub-filters for DCR-correctable coadds.

       More use cases and design ideas welcome.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              jbosch Jim Bosch
              Watchers:
              Christopher Waters, Eli Rykoff, Gabriele Comoretto [X] (Inactive), Gregory Dubois-Felsmann, Ian Sullivan, Jim Bosch, John Parejko, John Swinbank, Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              9 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Planned End:

                  Jenkins

                  No builds found.