search

dtcenter / METplus

Python scripting infrastructure for MET tools.
https://metplus.readthedocs.io
Apache License 2.0
98 stars 37 forks source link

Address Logging Statements requirement to specify Log Level #56

Open jimfrimel opened 7 years ago

jimfrimel commented 7 years ago

For example,

self.logger.info("INFO |[" + cur_function + ":" + cur_filename + "]" "| Finished extract tiles")

"INFO" should be present in the logger output line without having to specify it in the statement.

We should be able to do this ...

self.logger.info("[ + cur_function + ":" + cur_filename + "]" "| Finished extract tiles")

I'm pretty sure the log level and even the filename ... should be handled automatically by the produtil logging package ...

Revisit how we are setting up the handler in met_util.py ..

georgemccabe commented 5 years ago

We should also remove cur_filename because the logger outputs this information as well. It outputs the line number as well, so we really should remove cur_funtion since the line number should be enough information to track down an issue.