search

c3-time-domain / SeeChange

A time-domain data reduction pipeline (e.g., for handling images->lightcurves) for surveys like DECam and LS4
BSD 3-Clause "New" or "Revised" License
0 stars 4 forks source link

Rename and move _logger #227

Closed guynir42 closed 4 months ago

guynir42 commented 6 months ago

The ubiquitous _logger object should move out of models and into utils (not sure exactly where). We should also rename it to something else. I think logger is fine but maybe something else.

The underscore makes no sense as this is used widely outside its module by everyone else in the pipeline.

rknop commented 4 months ago

I've done this as part of the webap development and stress test. (I needed to modify the way we handled logging in subprocesses so I could see which one a given message was coming from, and moved the logger to util in so doing. )