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

Add instantiations in afw for meas_algorithms

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: afw
    • Labels:
      None
    • Team:
      Alert Production

      Description

      There are a few undefined symbols in meas_algorithms due to missing explicit instantiations in afw. I'll give this a shot.

        Attachments

          Issue Links

            Activity

            Hide
            jbosch Jim Bosch added a comment -

            I haven't been able to follow this too closely, but I'm a little nervous that we should be changing some compiler setting rather than changing our code. In any case, if this ever worked in the past (and it did), it means the template definitions that are missing now are at least available to meas_algorithms (i.e. they're in headers, not buried in a .cc file), so the question is why meas_algorithms is no longer instantiating them.

            If this does indeed do what we need to get the latest OS X working, then I don't have a problem with doing it, but we should open another fairly high-priority ticket to investigate this further and get to the bottom of it.

            Show
            jbosch Jim Bosch added a comment - I haven't been able to follow this too closely, but I'm a little nervous that we should be changing some compiler setting rather than changing our code. In any case, if this ever worked in the past (and it did), it means the template definitions that are missing now are at least available to meas_algorithms (i.e. they're in headers, not buried in a .cc file), so the question is why meas_algorithms is no longer instantiating them. If this does indeed do what we need to get the latest OS X working, then I don't have a problem with doing it, but we should open another fairly high-priority ticket to investigate this further and get to the bottom of it.
            Hide
            krughoff Simon Krughoff added a comment -

            I agree. I'm especially worried that we needed to export SchemaImpl methods since those should only ever be called internally to Schema (if I understand). I'll clone this one to a Summer 2014 task.

            Show
            krughoff Simon Krughoff added a comment - I agree. I'm especially worried that we needed to export SchemaImpl methods since those should only ever be called internally to Schema (if I understand). I'll clone this one to a Summer 2014 task.

              People

              Assignee:
              krughoff Simon Krughoff
              Reporter:
              krughoff Simon Krughoff
              Watchers:
              Jim Bosch, Simon Krughoff
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.