From 9b4e9cc8a0311e5243d69b73ed073e7ea441982e Mon Sep 17 00:00:00 2001
From: 游雁 <zhifu.gzf@alibaba-inc.com>
Date: 星期三, 27 三月 2024 16:05:29 +0800
Subject: [PATCH] train update
---
funasr/train_utils/trainer.py | 82 ++++++++++++++++++++++++++--------------
1 files changed, 53 insertions(+), 29 deletions(-)
diff --git a/funasr/train_utils/trainer.py b/funasr/train_utils/trainer.py
index 56ec604..116c9e3 100644
--- a/funasr/train_utils/trainer.py
+++ b/funasr/train_utils/trainer.py
@@ -71,21 +71,19 @@
self.use_ddp = use_ddp
self.use_fsdp = use_fsdp
self.device = kwargs.get('device', "cuda")
- self.avg_nbest_model = kwargs.get("avg_nbest_model", 5)
# self.kwargs = kwargs
self.log_interval = kwargs.get("log_interval", 50)
self.batch_total = 0
self.use_fp16 = use_fp16
- self.disable_gpu_cache = kwargs.get("disable_gpu_cache", True)
- # scaler = GradScaler(enabled=use_fp16) if use_fp16 else None
- # scaler = ShardedGradScaler(enabled=use_fp16) if use_fsdp else scaler
- # self.scaler = scaler
self.save_checkpoint_interval = kwargs.get("save_checkpoint_interval", 5000)
- self.keep_nbest_models = kwargs.get("keep_nbest_models", -1)
+ self.validate_interval = kwargs.get("validate_interval", 5000)
+ self.keep_nbest_models = kwargs.get("keep_nbest_models", 500)
+ self.avg_keep_nbest_models_type = kwargs.get("avg_keep_nbest_models_type", "acc")
+ self.avg_nbest_model = kwargs.get("avg_nbest_model", 5)
self.accum_grad = kwargs.get("accum_grad", 1)
self.grad_clip = kwargs.get("grad_clip", 10.0)
self.grad_clip_type = kwargs.get("grad_clip_type", 2.0)
- self.validate_interval = kwargs.get("validate_interval", 5000)
+
try:
@@ -103,8 +101,10 @@
self.val_loss_avg = 0.0
self.best_acc_idx = 0
self.saved_ckpts = {}
- self.val_acc_list = []
self.step_or_epoch = -1
+ self.best_step_or_epoch = ""
+ self.val_acc_step_or_eoch = {}
+ self.val_loss_step_or_eoch = {}
def save_checkpoint(self, epoch,
step=None,
@@ -124,14 +124,17 @@
if self.rank == 0:
logging.info(f"Save checkpoint: {epoch}, rank: {self.local_rank}\n")
- self.step_or_epoch += 1
+ # self.step_or_epoch += 1
state = {
'epoch': epoch,
'state_dict': model.state_dict(),
'optimizer': optim.state_dict(),
'scheduler': scheduler.state_dict(),
- "acc": self.val_acc_list,
- "step_or_epoch": self.step_or_epoch,
+ "saved_ckpts": self.saved_ckpts,
+ "val_acc_step_or_eoch": self.val_acc_step_or_eoch,
+ "val_loss_step_or_eoch": self.val_loss_step_or_eoch,
+ "best_step_or_epoch": self.best_step_or_epoch,
+ "avg_keep_nbest_models_type": slef.avg_keep_nbest_models_type,
}
if hasattr(model, "module"):
state["state_dict"] = model.module.state_dict()
@@ -150,23 +153,37 @@
logging.info(f'\nCheckpoint saved to {filename}\n')
latest = Path(os.path.join(self.output_dir, f'model.pt'))
torch.save(state, latest)
-
- if self.val_acc_list[self.step_or_epoch] >= self.val_acc_list[self.best_acc_idx]:
- self.best_acc_idx = self.step_or_epoch
- best_ckpt = Path(os.path.join(self.output_dir, f'model.pt.best'))
- torch.save(state, best_ckpt)
- logging.info(f"Update best acc: {self.val_acc_list[self.best_acc_idx]}, {best_ckpt}")
+ if self.best_step_or_epoch == "":
+ self.best_step_or_epoch = ckpt_name
+
+ if self.avg_keep_nbest_models_type == "acc":
+ if self.val_acc_step_or_eoch[ckpt_name] >= self.val_acc_step_or_eoch[self.best_step_or_epoch]:
+ self.best_step_or_epoch = ckpt_name
+ best_ckpt = Path(os.path.join(self.output_dir, f'model.pt.best'))
+ torch.save(state, best_ckpt)
+ logging.info(f"Update best acc: {self.val_acc_step_or_eoch[self.best_step_or_epoch]}, {best_ckpt}")
+ else:
+ logging.info(f"No improvement in acc: {self.val_acc_step_or_eoch[ckpt_name]} < {self.val_acc_step_or_eoch[self.best_step_or_epoch]}")
+ elif self.avg_keep_nbest_models_type == "loss":
+ if self.val_loss_step_or_eoch[ckpt_name] <= self.val_loss_step_or_eoch[self.best_step_or_epoch]:
+ self.best_step_or_epoch = ckpt_name
+ best_ckpt = Path(os.path.join(self.output_dir, f'model.pt.best'))
+ torch.save(state, best_ckpt)
+ logging.info(f"Update best loss: {self.val_loss_step_or_eoch[self.best_step_or_epoch]}, {best_ckpt}")
+ else:
+ logging.info(f"No improvement in loss: {self.val_loss_step_or_eoch[ckpt_name]} > {self.val_loss_step_or_eoch[self.best_step_or_epoch]}")
else:
- logging.info(f"No improvement in acc: {self.val_acc_list[self.best_acc_idx]}")
-
+ print("Undo")
+ self.saved_ckpts[ckpt_name] = getattr(self, f"val_{self.avg_keep_nbest_models_type}_step_or_eoch")[ckpt_name]
if self.keep_nbest_models > 0:
- self.saved_ckpts[ckpt_name] = self.val_acc_list[-1]
if len(self.saved_ckpts) > self.keep_nbest_models:
-
- min_key = min(self.saved_ckpts, key=self.saved_ckpts.get)
- if min_key in self.saved_ckpts:
- del self.saved_ckpts[min_key]
- filename = os.path.join(self.output_dir, min_key)
+ if self.avg_keep_nbest_models_type == "acc":
+ key = min(self.saved_ckpts, key=self.saved_ckpts.get)
+ else:
+ key = max(self.saved_ckpts, key=self.saved_ckpts.get)
+ if key in self.saved_ckpts:
+ del self.saved_ckpts[key]
+ filename = os.path.join(self.output_dir, key)
logging.info(f"Delete: {filename}")
if os.path.exists(filename):
os.remove(filename)
@@ -213,8 +230,10 @@
if scaler is not None and 'scaler_state' in checkpoint:
scaler.load_state_dict(checkpoint['scaler_state'])
- self.val_acc_list = checkpoint["acc"]
- self.step_or_epoch = checkpoint["step_or_epoch"]
+ self.saved_ckpts = checkpoint["saved_ckpts"]
+ self.val_acc_step_or_eoch = checkpoint["val_acc_step_or_eoch"] if "val_acc_step_or_eoch" in checkpoint else {}
+ self.val_loss_step_or_eoch = checkpoint["val_loss_step_or_eoch"] if "val_loss_step_or_eoch" in checkpoint else {}
+ self.val_loss_step_or_eoch = checkpoint["best_step_or_epoch"] if "best_step_or_epoch" in checkpoint else ""
model.to(self.device)
print(f"Checkpoint loaded successfully from '{ckpt}'")
else:
@@ -458,8 +477,13 @@
if self.use_ddp or self.use_fsdp:
iterator_stop.fill_(1)
dist.all_reduce(iterator_stop, dist.ReduceOp.SUM)
-
- self.val_acc_list.append(self.val_acc_avg)
+
+ if kwargs.get("step", None) is None:
+ ckpt_name = f'model.pt.ep{epoch}'
+ else:
+ ckpt_name = f'model.pt.ep{epoch}.{kwargs.get("step")}'
+ self.val_acc_step_or_eoch[ckpt_name] = self.val_acc_avg
+ self.val_loss_step_or_eoch[ckpt_name] = self.val_loss_avg
model.train()
if self.use_ddp or self.use_fsdp:
--
Gitblit v1.9.1