Address remaining PR review comments: add docstring entries for validation errors and extend tests

- Update SoftclDiceLoss docstring to document TypeError for iter_ type check and ValueError for negative iter_
- Update SoftDiceclDiceLoss docstring to document ValueError for alpha range validation
- Add test_invalid_iter_type to verify TypeError when iter_ is not an int
- Add test_invalid_iter_value to verify ValueError when iter_ is negative
- Add test_invalid_alpha and test_invalid_alpha_negative to verify alpha validation

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

Author: aymuos15

monai/losses/cldice.py

@@ -152,6 +152,8 @@ def __init__(
 
         Raises:
             TypeError: When ``other_act`` is not an ``Optional[Callable]``.
+            TypeError: When ``iter_`` is not an ``int``.
+            ValueError: When ``iter_`` is a negative integer.
             ValueError: When more than 1 of [``sigmoid=True``, ``softmax=True``, ``other_act is not None``].
                 Incompatible values.
 
@@ -296,6 +298,7 @@ def __init__(
 
         Raises:
             TypeError: When ``other_act`` is not an ``Optional[Callable]``.
+            ValueError: When ``alpha`` is not in ``[0, 1]``.
             ValueError: When more than 1 of [``sigmoid=True``, ``softmax=True``, ``other_act is not None``].
                 Incompatible values.
 

tests/losses/test_cldice_loss.py

@@ -106,6 +106,14 @@ def test_invalid_other_act(self):
         with self.assertRaises(TypeError):
             SoftclDiceLoss(other_act="invalid")
 
+    def test_invalid_iter_type(self):
+        with self.assertRaises(TypeError):
+            SoftclDiceLoss(iter_=3.0)
+
+    def test_invalid_iter_value(self):
+        with self.assertRaises(ValueError):
+            SoftclDiceLoss(iter_=-1)
+
 
 class TestSoftDiceclDiceLoss(unittest.TestCase):
     @parameterized.expand(COMBINED_CASES)
@@ -131,6 +139,14 @@ def test_channel_mismatch(self):
         with self.assertRaises(ValueError):
             loss(torch.ones(2, 3, 8, 8), torch.ones(2, 2, 8, 8))
 
+    def test_invalid_alpha(self):
+        with self.assertRaises(ValueError):
+            SoftDiceclDiceLoss(alpha=1.5)
+
+    def test_invalid_alpha_negative(self):
+        with self.assertRaises(ValueError):
+            SoftDiceclDiceLoss(alpha=-0.5)
+
 
 if __name__ == "__main__":
     unittest.main()