-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add logging #17
Comments
Clarification, these are log messages e.g. to a terminal about what is going on internally? |
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. |
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. |
@cberys do you want more logging message in specific places? |
@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? |
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.
The text was updated successfully, but these errors were encountered: