-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Chapter08 python file - reporthook function code fix to prevent div by 0 error #123
base: master
Are you sure you want to change the base?
Conversation
Check out this pull request on You'll be able to see Jupyter notebook diff and discuss changes. Powered by ReviewNB. |
Hello @teamiceberg! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:
|
Dear author:
The function fails due to div by zero error as duration = 0 initially. I did a simple code change to accommodate this bug. If you find it useful, please merge change.
Also, ignore all my other annotations change. That is for my own learning purposes.
Thank you.