Unverified Commit 16271080 authored by Sai-Suraj-27's avatar Sai-Suraj-27 Committed by GitHub
Browse files

fix: Added missing raise keyword for few exceptions (#32333)

Fixed raising of few exceptions.
parent bd54ed2e
......@@ -98,7 +98,7 @@ def regularization(model: nn.Module, mode: str):
elif mode == "l0":
regu += torch.sigmoid(param - 2 / 3 * np.log(0.1 / 1.1)).sum() / param.numel()
else:
ValueError("Don't know this mode.")
raise ValueError("Don't know this mode.")
counter += 1
return regu / counter
......
......@@ -101,7 +101,7 @@ def regularization(model: nn.Module, mode: str):
elif mode == "l0":
regu += torch.sigmoid(param - 2 / 3 * np.log(0.1 / 1.1)).sum() / param.numel()
else:
ValueError("Don't know this mode.")
raise ValueError("Don't know this mode.")
counter += 1
return regu / counter
......
......@@ -744,7 +744,7 @@ class WandbLogModel(str, Enum):
if not isinstance(value, str):
raise ValueError(f"Expecting to have a string `WANDB_LOG_MODEL` setting, but got {type(value)}")
if value.upper() in ENV_VARS_TRUE_VALUES:
DeprecationWarning(
raise DeprecationWarning(
f"Setting `WANDB_LOG_MODEL` as {os.getenv('WANDB_LOG_MODEL')} is deprecated and will be removed in "
"version 5 of transformers. Use one of `'end'` or `'checkpoint'` instead."
)
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment