Rework logging functionality. [Issue/66]

AdamOpps commented 6 months ago

Logs to 'newarenda' instead of 'root', which is standard practice for modules.
Includes various error messages and warnings to appropriate log levels
log_level is a parameter that can be passed to read(). It is passable in the CLI as well, from a set of choices. Default option is 'INFO' to mimic previous functionality. This might want to be changed to 'NOTSET' in the future as it will default to the parent's logging level.

NOTE: Moving forward, all log messages should be called from "logger". Example: logger.info('Good! This message is saved to name: newarenda) logging.info('Bad! This message is saved to name: root')

d-cogswell commented 6 months ago

Hi, I'm finally getting caught up here. This is a nice improvement! I noticed there are still some logging happening to root (the old way) for 'Active mass' and 'Remarks'. These should be changed to newarenda.

d-cogswell commented 6 months ago

I'm going to merge #68 and #70 while we continue working on this. I've already encountered the #68 issue in my own data processing.

AdamOpps commented 6 months ago

Sounds good. Sorry for my delay, I have been dealing with some personal things that have set me back a bit.

Solid-Energy-Systems / NewareNDA

Rework logging functionality. [Issue/66] #71