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

Provide a way to merge multiple afwTable catalogs with identical schema

    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
            price Paul Price added a comment -

            See afw/examples/catTables.py

            Show
            price Paul Price added a comment - See afw/examples/catTables.py
            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
            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.

              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:

                  Summary Panel