ACCESS-Community-Hub / ACCESS-MOPPeR

A tool to post-process ACCESS Model output using CMOR, based on APP4, work in progress!
https://access-mopper.readthedocs.io/en/latest/
Apache License 2.0
0 stars 0 forks source link

rationalise calls to logs #147

Closed paolap closed 2 days ago

paolap commented 3 days ago

I'm currently passing the logger explicitly but this shouldn't be needed all is needed is the logger name.

paolap commented 3 days ago

First part of this is implemented for mop_log and mopdb_log and they both work correctly. Including when called in test.s Need to update var_log, in this case while we don't need anymore to pass the logger around in ctx we still need to pass the name around as each process uses a different file handle. This is also now implemented, NB I had to add a "logger.propagate = False" in their configuration otherwise each message was duplicated in job output. I use _log as name for the loggers.