desihub / specex

DESI spectrograph PSF fitting
BSD 3-Clause "New" or "Revised" License
0 stars 4 forks source link

Remove boost::program_options #47

Closed marcelo-alvarez closed 3 years ago

marcelo-alvarez commented 3 years ago

Intermediate step (rm_poptions) in sequence of branches to remove dependence on HARP and BOOST. input_refactor --> rm_harp --> rm_unused --> matrix_refactor --> rm_poptions

This pull request removes all dependence on boost::program_options and instead uses getopt_long(). This version has no dependence on HARP or BOOST.

Performance is unchanged and the code produces identical bitwise results for the file fit-psf-b1-00068217.fits with respect to master when running desi_compute_psf --input-image /global/cfs/cdirs/desi/spectro/redux/blanc/preproc/20201216/00068217/preproc-b1-00068217.fits --input-psf /global/cfs/cdirs/desi/spectro/redux/blanc/exposures/20201216/00068217/shifted-input-psf-b1-00068217.fits --output-psf ./fit-psf-b1-00068217.fits

on cori.