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

Remove boost dependencies where possible

    XMLWordPrintable

    Details

    • Type: Improvement
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: None
    • Labels:
      None

      Description

      In X16/DM-5580, we removed Boost from a number of packages. However, we may not have rigorously updated their dependency lists to indicate where Boost is no longer required. Please do so.

        Attachments

          Issue Links

            Activity

            Hide
            tjenness Tim Jenness added a comment -

            I have done a back of the envelope audit:

            Boost listed as dependency for:

            base
            cat
            ctrl_events
            galsim
            log
            lsst_thirdparty
            meas_extensions_psfex
            ndarray
            partition
            pex_exceptions
            qserv
            templates
            utils

            Actually used in:

            afw
            coadd_utils (just boost::format)
            ctrl_events
            daf_base
            daf_persistence
            ip_diffim
            jointcal
            log (in a single test that uses boost:format)
            meas_algorithms
            meas_astrom
            meas_base
            meas_extensions_photometryKron
            meas_extensions_psfex
            meas_extensions_shapeHSM
            meas_modelfit
            mops_daymops
            ndarray
            obs_subaru
            partition
            pex_exceptions
            pex_logging
            pex_policy
            qserv
            shapelet
            utils

            So package that should not be saying boost but are:

            base
            cat
            galsim

            and there are many packages that are using boost but don't declare that dependency.

            Show
            tjenness Tim Jenness added a comment - I have done a back of the envelope audit: Boost listed as dependency for: base cat ctrl_events galsim log lsst_thirdparty meas_extensions_psfex ndarray partition pex_exceptions qserv templates utils Actually used in: afw coadd_utils (just boost::format) ctrl_events daf_base daf_persistence ip_diffim jointcal log (in a single test that uses boost:format) meas_algorithms meas_astrom meas_base meas_extensions_photometryKron meas_extensions_psfex meas_extensions_shapeHSM meas_modelfit mops_daymops ndarray obs_subaru partition pex_exceptions pex_logging pex_policy qserv shapelet utils So package that should not be saying boost but are: base cat galsim and there are many packages that are using boost but don't declare that dependency.
            Hide
            jbosch Jim Bosch added a comment - - edited

            Matthias Wittgen, here's another old ticket that's tangentially related to a new one (or at least one we've talked about creating). It's actually more about tracking where boost is used, not removing boost usage, but we should do both whenever we take a serious stab at replacing any boost usage with new std stuff or our own implementations anywhere. Not a priority at all, but something to maybe check off as you do other work anyway.

            Show
            jbosch Jim Bosch added a comment - - edited Matthias Wittgen , here's another old ticket that's tangentially related to a new one (or at least one we've talked about creating). It's actually more about tracking where boost is used, not removing boost usage, but we should do both whenever we take a serious stab at replacing any boost usage with new std stuff or our own implementations anywhere. Not a priority at all, but something to maybe check off as you do other work anyway.
            Hide
            wittgen Matthias Wittgen added a comment - - edited

            Switch from boost::variant to std::variant already is available in u/jbosch/c++17.
            boost::filesystem. to std::filesystem should be straight forward. Moving to std::regex requires some more carful considerations.Look into replacement of boost timer.

            Show
            wittgen Matthias Wittgen added a comment - - edited Switch from boost::variant to std::variant already is available in u/jbosch/c++17 . boost::filesystem . to std::filesystem should be straight forward. Moving to std::regex requires some more carful considerations.Look into replacement of boost timer.
            Hide
            wittgen Matthias Wittgen added a comment -

            Move the above C++17 improvement to DM-29563.

            Show
            wittgen Matthias Wittgen added a comment - Move the above C++17 improvement to DM-29563 .

              People

              Assignee:
              wittgen Matthias Wittgen
              Reporter:
              swinbank John Swinbank
              Watchers:
              Eli Rykoff, Jim Bosch, Matthias Wittgen, Tim Jenness
              Votes:
              1 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins Builds

                  No builds found.