search

cchdo / hydro

The big ol CCHDO netCDF-CF project
https://hydro.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
1 stars 2 forks source link

Add logging #17

Open cberys opened 3 years ago

cberys commented 3 years ago

Add logging specifically for changes made to files on read in, but I would guess also in general. Please edit or add to this issue.

DocOtak commented 3 years ago

Clarification, these are log messages e.g. to a terminal about what is going on internally?

cberys commented 3 years ago

I think yes for the use stated, but there might have been other uses as well. I have a note from the meeting to make this issue but I wasn't sure on the specifics.

DocOtak commented 2 years ago

Lots of debug logging in the fast reader branch so far, so this will definitely happen as I just need to leave them all in rather than remove. It's using the python logging module, so you need to attach a logging handler to it to get output if using the API directly.

DocOtak commented 2 months ago

@cberys do you want more logging message in specific places?

cberys commented 2 months ago

@DocOtak I believe there has been a ton of improvement to error messages for the user since this issue was made.(I think in the beginning there was basically none or very limited) I don't know if you want to close the issue or leave it open for when specific requests come up and rename to "logging wish list" or whatever - up to you.

Additionally - maybe create some instructionals on using the logging module and what is available?