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

Wrap coadd_chisquared with pybind11

    Details

      Attachments

        Issue Links

          Activity

          Hide
          rowen Russell Owen added a comment -

          A trivial wrapper

          Show
          rowen Russell Owen added a comment - A trivial wrapper
          Hide
          krzys Krzysztof Findeisen added a comment -

          Looks good. I'm puzzled by the use of double rather than single quotes in the scons script, but that's it.

          Show
          krzys Krzysztof Findeisen added a comment - Looks good. I'm puzzled by the use of double rather than single quotes in the scons script, but that's it.
          Hide
          rowen Russell Owen added a comment -

          Thank you for the review. I stick to double quotes in the SConscript file, but if folks really care we can change it.

          I started a Jenkins build but it seems to have vanished (it doesn't show up on my page of Jenkins builds). I'll merge for now and hope for the best. It works on OS X.

          Show
          rowen Russell Owen added a comment - Thank you for the review. I stick to double quotes in the SConscript file, but if folks really care we can change it. I started a Jenkins build but it seems to have vanished (it doesn't show up on my page of Jenkins builds). I'll merge for now and hope for the best. It works on OS X.

            People

            • Assignee:
              rowen Russell Owen
              Reporter:
              pschella Pim Schellart [X] (Inactive)
              Reviewers:
              Krzysztof Findeisen
              Watchers:
              Krzysztof Findeisen, Pim Schellart [X] (Inactive), Russell Owen
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Summary Panel