address coderabbit comments

Signed-off-by: Soumya Snigdha Kundu <soumya_snigdha.kundu@kcl.ac.uk>

Author: aymuos15

monai/losses/cldice.py

@@ -161,6 +161,8 @@ def __init__(
             raise TypeError(f"other_act must be None or callable but is {type(other_act).__name__}.")
         if int(sigmoid) + int(softmax) + int(other_act is not None) > 1:
             raise ValueError("Incompatible values: more than 1 of [sigmoid=True, softmax=True, other_act is not None].")
+        if not isinstance(iter_, int):
+            raise TypeError(f"iter_ must be an integer but got {type(iter_).__name__}.")
         if iter_ < 0:
             raise ValueError(f"iter_ must be a non-negative integer but got {iter_}.")
         self.iter = iter_
@@ -299,6 +301,8 @@ def __init__(
 
         """
         super().__init__()
+        if not 0.0 <= alpha <= 1.0:
+            raise ValueError(f"alpha must be in [0, 1] but got {alpha}.")
         self.dice = DiceLoss(
             include_background=include_background,
             to_onehot_y=False,

tests/losses/test_cldice_loss.py

@@ -72,7 +72,6 @@
 
 
 class TestSoftclDiceLoss(unittest.TestCase):
-
     @parameterized.expand(CLDICE_CASES)
     def test_result(self, loss_params, input_data, expected_val):
         loss = SoftclDiceLoss(**loss_params)
@@ -109,7 +108,6 @@ def test_invalid_other_act(self):
 
 
 class TestSoftDiceclDiceLoss(unittest.TestCase):
-
     @parameterized.expand(COMBINED_CASES)
     def test_result(self, loss_params, input_data, expected_val):
         loss = SoftDiceclDiceLoss(**loss_params)