-
Notifications
You must be signed in to change notification settings - Fork 184
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
【benchmark】fix gpu_mem unit #678
Conversation
Thanks for your contribution! |
ppsci/solver/printer.py
Outdated
@@ -74,18 +74,17 @@ def log_train_info( | |||
(trainer.epochs - epoch_id + 1) * trainer.iters_per_epoch - iter_id | |||
) * trainer.train_time_info["batch_cost"].avg | |||
eta_msg = f"eta: {str(datetime.timedelta(seconds=int(eta_sec))):s}" | |||
|
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.
换行不要删除
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.
已添加
ppsci/solver/printer.py
Outdated
@@ -74,18 +74,18 @@ def log_train_info( | |||
(trainer.epochs - epoch_id + 1) * trainer.iters_per_epoch - iter_id | |||
) * trainer.train_time_info["batch_cost"].avg | |||
eta_msg = f"eta: {str(datetime.timedelta(seconds=int(eta_sec))):s}" | |||
|
|||
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.
多余空格删除
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.
LGTM
* add max_mem_reserved for benchmark * add max_mem_reserved for benchamrk * Update printer.py * Update printer.py
PR types
Others
PR changes
Others
Describe
修改gpu_mem 单位B->MB