desihub / specex

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

Matrix refactor #46

Closed marcelo-alvarez closed 3 years ago

marcelo-alvarez commented 3 years ago

Intermediate step (matrix_refactor) 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::numeric::ublas through the new classes matrix, matrix_double and matrix_mask.

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.