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

Add variadic versions of applyForward and applyInverse that take N vectors

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: astshim

      Description

      Consider adding a variadic version of Mapping.applyForward and applyInverse that takes a separate vector for each axis. AST works this way internally, so it could be implemented without copying data, and it would allow data where each axis comes from from different column of a catalog to be transformed.

      The one use case we have right now is the vector version of TransformBoundedField::evaluate. Presently it has to copy its inputs, but with this ticket it could avoid making the copy.

      I'm not convinced this is worth the work, but this ticket serves as a placeholder in case we decide to do it.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            I'm just going to call this not worth the effort (seemed dubious even at issue creation); the important interfaces these days are the Python ones, and we can't pybind11-wrap a variadic C++ method fully anyway.

            Show
            jbosch Jim Bosch added a comment - I'm just going to call this not worth the effort (seemed dubious even at issue creation); the important interfaces these days are the Python ones, and we can't pybind11-wrap a variadic C++ method fully anyway.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              rowen Russell Owen
              Watchers:
              Jim Bosch, Russell Owen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.