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

Update matching routine to handle string columns

    XMLWordPrintable

    Details

      Description

      The current matching routine in faro cannot have string-valued columns. Update it to make this possible, so we can include things like filter names in the matched catalogs.

        Attachments

          Issue Links

          blocks

          Story - Created by Jira Software - do not edit or delete. Issue type for a user story. DM-26674 Add metadata into matched catalog columns

          • Done

            Activity

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            jcarlin Jeffrey Carlin added a comment -

            Simon Krughoff Is this something we still need to do? For now we've implemented (see DM-26674 and this commit) the workaround of assigning numerical codes to filters in a dict, and other metadata such as visit and ccd can be passed as ints as well.

            I'm inclined to close this as "Won't Fix", but am interested to hear your thoughts.

            Show
            jcarlin Jeffrey Carlin added a comment - Simon Krughoff Is this something we still need to do? For now we've implemented (see DM-26674 and this commit ) the workaround of assigning numerical codes to filters in a dict, and other metadata such as visit and ccd can be passed as ints as well. I'm inclined to close this as "Won't Fix", but am interested to hear your thoughts.
            Hide
            Permalink
            lguy Leanne Guy added a comment -

            If the current workaround is doing what we need, I'd like to de-prioritise this. Simon Krughoff Jeffrey Carlin?

            Show
            lguy Leanne Guy added a comment - If the current workaround is doing what we need, I'd like to de-prioritise this. Simon Krughoff Jeffrey Carlin ?
            Hide
            Permalink
            lguy Leanne Guy added a comment -

            No longer relevant 

            Show
            lguy Leanne Guy added a comment - No longer relevant 

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              jcarlin Jeffrey Carlin
              Watchers:
              Jeffrey Carlin, Leanne Guy
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.