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

Provide a way to merge multiple afwTable catalogs with identical schema

    XMLWordPrintable

    Details

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

      Description

      Concatenating afwTable catalogs to create one catalog with contiguous memory isn't quite as simple as it should be. One reason is that the reserve() member doesn't do what you might expect as it makes no guarantee of contiguous storage [this is why std::deque::reserve doesn't exist].

      We should provide a utility function that does this right (Jim tells me that there is a right way – I know a variety of ways that work that don't quite work when you allow for things like slots, or that require a deep copy of the result)

        Attachments

          Issue Links

            Activity

            Hide
            swinbank John Swinbank added a comment -

            This is not quite the same as DM-820, but the implementation will be covered as part of that ticket.

            Show
            swinbank John Swinbank added a comment - This is not quite the same as DM-820 , but the implementation will be covered as part of that ticket.
            Hide
            rhl Robert Lupton added a comment -

            I did, and it's not the best way to do it. And this functionality shouldn't be on examples.

            Show
            rhl Robert Lupton added a comment - I did, and it's not the best way to do it. And this functionality shouldn't be on examples.
            Hide
            price Paul Price added a comment -

            See afw/examples/catTables.py

            Show
            price Paul Price added a comment - See afw/examples/catTables.py

              People

              Assignee:
              jbosch Jim Bosch
              Reporter:
              rhl Robert Lupton
              Watchers:
              John Swinbank, Paul Price, Robert Lupton
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: