LSSTDESC / rail_bpz

RAIL interface to BPZ algorithms
MIT License
0 stars 1 forks source link

Remove .columns file dependence #53

Closed sschmidt23 closed 1 month ago

sschmidt23 commented 1 month ago

Having to read in a few numbers from a .columns file rather than via a config param is annoying, and a remnant of how DESC_BPZ does things. It should be simple to remove the one or two things that are read from .columns and instead make those config params.

sschmidt23 commented 1 month ago

Looking at the code, it's even less involved than I thought, I thought that we read the names of the mags, mag_errs, and zp_offsets from the columns file, but I already moved those to config params, the only remaining thing in the .columns is the filter names, should be trivial to move that to a config param.

sschmidt23 commented 1 month ago

closed by #54