NCAR / ADF

A unified collection of python scripts used to generate standard plots from CAM outputs.
Creative Commons Attribution 4.0 International
35 stars 29 forks source link

Thoughts on increasing stdout/err to include paths? #265

Open bitterbark opened 10 months ago

bitterbark commented 10 months ago

New feature type

Other (please explain below)

What is this new feature?

I'd like to have directory path printed whenever something is successful or not successful. But I know I tend to like more output than most people, so thought I would check.

I believe it would save me time when the package works, because I can check the output file and it would say (for example): ...Zonal mean plots have been generated successfully. See and then I can just go look there. Otherwise I have to check my config file for where I said they would go, and replace variables etc.

Even more useful would be to say what directory is being looked in when files aren't found. I believe this would have saved me a few hours of debugging today because I had to go into the code and print out what location it was looking in. Yes, this took too long but I hadn't looked in that code before. (BTW I had failed to set the correct path of my baseline climo files).

I can implement this if approved.

Assistance required?

No, I will make a PR when the feature is ready

Extra info

No response