-
Notifications
You must be signed in to change notification settings - Fork 462
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
fix: Adds missing console log and removed Tensorboard #626
Conversation
Codecov Report
@@ Coverage Diff @@
## main #626 +/- ##
==========================================
- Coverage 96.18% 96.16% -0.03%
==========================================
Files 111 111
Lines 4300 4300
==========================================
- Hits 4136 4135 -1
- Misses 164 165 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
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.
thanks!
@@ -11,8 +11,6 @@ | |||
import datetime | |||
import hashlib | |||
import time | |||
from collections import deque | |||
from pathlib import Path | |||
|
|||
import numpy as np | |||
import tensorflow as tf |
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.
May be we also have to import wandb in the script?
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.
but it is on line19 ;)
This PR introduces the following modifications:
Any feedback is welcome!