-
Notifications
You must be signed in to change notification settings - Fork 0
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
Issue/#29/optimize console output #39
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice.
Just one comment regarding the traceback print message.
1abc2ab
to
6071458
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, I noticed that the `logging_configuration.py' file is not consistent with the rest of the codebase (usually we put different classes and functions in separate files). Consider moving logging to its own subdirectory and splitting it into different files.
Also this seems to be a duplicate of #42. If this was a mistake you can simply revert your changes here and apply requested changes on the quickfix branch. |
implementation of progress in processor
This reverts commit 76ec379.
a6de29e
to
29806db
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
YOLO
Added more descriptive information to output while reading and processing files.