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

Use standard smart pointers in ndarray

    XMLWordPrintable

    Details

    • Type: Story
    • Status: Won't Fix
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ndarray
    • Labels:
      None
    • Team:
      Data Release Production

      Description

      Most of the stack has now been converted to use std::shared_ptr and friends, but ndarray has so far been excluded because of it's extensive use of boost::intrusive_ptr.
      This ticket aims to fix it.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            Did we decide about upstream ndarray? I have an open PR there for local LSST modifications. Can we at least ensure that the patches here are added to that pull request?

            Show
            tjenness Tim Jenness added a comment - Did we decide about upstream ndarray? I have an open PR there for local LSST modifications. Can we at least ensure that the patches here are added to that pull request?
            Hide
            jbosch Jim Bosch added a comment -

            I don't expect to do this work until we have synced with external ndarray and switched to it. I'd also like to do a stable release and some other major work on ndarray on (external) master after that, but before switching away from intrusive_ptr.

            Show
            jbosch Jim Bosch added a comment - I don't expect to do this work until we have synced with external ndarray and switched to it. I'd also like to do a stable release and some other major work on ndarray on (external) master after that, but before switching away from intrusive_ptr.
            Hide
            tjenness Tim Jenness added a comment -

            Jim Bosch is this ticket still relevant?

            Show
            tjenness Tim Jenness added a comment - Jim Bosch is this ticket still relevant?
            Hide
            jbosch Jim Bosch added a comment -

            The work has not been done, but I have no plans to do it unless there's a concrete need for it (e.g. dropping boost looks otherwise viable).

            Show
            jbosch Jim Bosch added a comment - The work has not been done, but I have no plans to do it unless there's a concrete need for it (e.g. dropping boost looks otherwise viable).
            Hide
            jbosch Jim Bosch added a comment -

            Situation hasn't changed appreciably since the last CCB review, except that I'm now less optimistic about personally ever getting very low-priority things. Closing as Won't Fix.

            Show
            jbosch Jim Bosch added a comment - Situation hasn't changed appreciably since the last CCB review, except that I'm now less optimistic about personally ever getting very low-priority things. Closing as Won't Fix.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              pschella Pim Schellart [X] (Inactive)
              Watchers:
              Jim Bosch, John Swinbank, Pim Schellart [X] (Inactive), Tim Jenness
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.