search

desihub / specex

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

Add prefix to log messages #61

Closed julienguy closed 2 years ago

julienguy commented 2 years ago

Add prefix to log messages with camera,expid,bundle to be able to separate those from different processes running different bundles.

Example:

INFO z3-00074755-15: Starting FitSeveralSpots TAIL&CONTINUUM
INFO z3-00074755-06: specex::PSF_Fitter::FitSeveralSpots dchi2=9.4589 chi2pdf = 1.63103 npar = 1135
INFO z3-00074755-03: specex::PSF_Fitter::FitSeveralSpots inc. signal in w=0, npix footprint = 54922
INFO z3-00074755-15: WEIGHTS: Set weight to ZERO at the CORE of spots (to fit tails or continuum)
INFO z3-00074755-15: specex::PSF_Fitter::FitSeveralSpots inc. signal in w=0, npix footprint = 615311
INFO z3-00074755-04: specex::PSF_Fitter::FitSeveralSpots dchi2=0.0133823 chi2pdf = 1.65984 npar = 1130
INFO z3-00074755-12: specex::PSF_Fitter::FitSeveralSpots dchi2=0.0018115 chi2pdf = 1.77947 npar = 1133
julienguy commented 2 years ago

This PR is ready for review.

sbailey commented 2 years ago

These changes look fine to me and I have independently tested compiling and running with this.

@marcelo-alvarez please do any other checks/comments that you would like, and then take care of final merge, making a new tag 0.8.4, and installing. Thanks.