part of #8421

remove deprecated functionality

Signed-off-by: YunLiu <[email protected]>

Author: KumoLiu

docs/source/metrics.rst

@@ -98,7 +98,6 @@ Metrics
 `Hausdorff distance`
 --------------------
 .. autofunction:: compute_hausdorff_distance
-.. autofunction:: compute_percent_hausdorff_distance
 
 .. autoclass:: HausdorffDistanceMetric
     :members:

monai/bundle/scripts.py

@@ -629,9 +629,6 @@ def download(
     _check_monai_version(bundle_dir_, name_)
 
 
-@deprecated_arg("net_name", since="1.2", removed="1.5", msg_suffix="please use ``model`` instead.")
-@deprecated_arg("net_kwargs", since="1.2", removed="1.5", msg_suffix="please use ``model`` instead.")
-@deprecated_arg("return_state_dict", since="1.2", removed="1.5")
 def load(
     name: str,
     model: torch.nn.Module | None = None,
@@ -650,10 +647,6 @@ def load(
     workflow_name: str | BundleWorkflow | None = None,
     args_file: str | None = None,
     copy_model_args: dict | None = None,
-    return_state_dict: bool = True,
-    net_override: dict | None = None,
-    net_name: str | None = None,
-    **net_kwargs: Any,
 ) -> object | tuple[torch.nn.Module, dict, dict] | Any:
     """
     Load model weights or TorchScript module of a bundle.
@@ -699,11 +692,6 @@ def load(
         workflow_name: specified bundle workflow name, should be a string or class, default to "ConfigWorkflow".
         args_file: a JSON or YAML file to provide default values for all the args in "download" function.
         copy_model_args: other arguments for the `monai.networks.copy_model_state` function.
-        return_state_dict: whether to return state dict, if True, return state_dict, else a corresponding network
-            from `_workflow.network_def` will be instantiated and load the achieved weights.
-        net_override: id-value pairs to override the parameters in the network of the bundle, default to `None`.
-        net_name: if not `None`, a corresponding network will be instantiated and load the achieved weights.
-            This argument only works when loading weights.
         net_kwargs: other arguments that are used to instantiate the network class defined by `net_name`.
 
     Returns:
@@ -719,9 +707,6 @@ def load(
             when `model` and `net_name` are all `None`.
 
     """
-    if return_state_dict and (model is not None or net_name is not None):
-        warnings.warn("Incompatible values: model and net_name are all specified, return state dict instead.")
-
     bundle_dir_ = _process_bundle_dir(bundle_dir)
     net_override = {} if net_override is None else net_override
     copy_model_args = {} if copy_model_args is None else copy_model_args
@@ -757,11 +742,8 @@ def load(
         warnings.warn(f"the state dictionary from {full_path} should be a dictionary but got {type(model_dict)}.")
         model_dict = get_state_dict(model_dict)
 
-    if return_state_dict:
-        return model_dict
-
     _workflow = None
-    if model is None and net_name is None:
+    if model is None:
         bundle_config_file = bundle_dir_ / name / "configs" / f"{workflow_type}.json"
         if bundle_config_file.is_file():
             _net_override = {f"network_def#{key}": value for key, value in net_override.items()}
@@ -781,10 +763,6 @@ def load(
                 return model_dict
             else:
                 model = _workflow.network_def
-    elif net_name is not None:
-        net_kwargs["_target_"] = net_name
-        configer = ConfigComponent(config=net_kwargs)
-        model = configer.instantiate()  # type: ignore
 
     model.to(device)  # type: ignore
 

monai/bundle/workflows.py

@@ -45,10 +45,6 @@ class BundleWorkflow(ABC):
             or "infer", "inference", "eval", "evaluation" for a inference workflow,
             other unsupported string will raise a ValueError.
             default to `None` for only using meta properties.
-        workflow: specifies the workflow type: "train" or "training" for a training workflow,
-            or "infer", "inference", "eval", "evaluation" for a inference workflow,
-            other unsupported string will raise a ValueError.
-            default to `None` for common workflow.
         properties_path: the path to the JSON file of properties. If `workflow_type` is specified, properties will be
             loaded from the file based on the provided `workflow_type` and meta. If no `workflow_type` is specified,
             properties will default to loading from "meta". If `properties_path` is None, default properties
@@ -65,17 +61,9 @@ class BundleWorkflow(ABC):
     supported_train_type: tuple = ("train", "training")
     supported_infer_type: tuple = ("infer", "inference", "eval", "evaluation")
 
-    @deprecated_arg(
-        "workflow",
-        since="1.2",
-        removed="1.5",
-        new_name="workflow_type",
-        msg_suffix="please use `workflow_type` instead.",
-    )
     def __init__(
         self,
         workflow_type: str | None = None,
-        workflow: str | None = None,
         properties_path: PathLike | None = None,
         meta_file: str | Sequence[str] | None = None,
         logging_file: str | None = None,
@@ -102,7 +90,6 @@ def __init__(
                         )
                         meta_file = None
 
-        workflow_type = workflow if workflow is not None else workflow_type
         if workflow_type is not None:
             if workflow_type.lower() in self.supported_train_type:
                 workflow_type = "train"
@@ -403,10 +390,6 @@ class ConfigWorkflow(BundleWorkflow):
             or "infer", "inference", "eval", "evaluation" for a inference workflow,
             other unsupported string will raise a ValueError.
             default to `None` for common workflow.
-        workflow: specifies the workflow type: "train" or "training" for a training workflow,
-            or "infer", "inference", "eval", "evaluation" for a inference workflow,
-            other unsupported string will raise a ValueError.
-            default to `None` for common workflow.
         properties_path: the path to the JSON file of properties. If `workflow_type` is specified, properties will be
             loaded from the file based on the provided `workflow_type` and meta. If no `workflow_type` is specified,
             properties will default to loading from "train". If `properties_path` is None, default properties
@@ -419,13 +402,6 @@ class ConfigWorkflow(BundleWorkflow):
 
     """
 
-    @deprecated_arg(
-        "workflow",
-        since="1.2",
-        removed="1.5",
-        new_name="workflow_type",
-        msg_suffix="please use `workflow_type` instead.",
-    )
     def __init__(
         self,
         config_file: str | Sequence[str],
@@ -436,11 +412,9 @@ def __init__(
         final_id: str = "finalize",
         tracking: str | dict | None = None,
         workflow_type: str | None = "train",
-        workflow: str | None = None,
         properties_path: PathLike | None = None,
         **override: Any,
     ) -> None:
-        workflow_type = workflow if workflow is not None else workflow_type
         if config_file is not None:
             _config_files = ensure_tuple(config_file)
             config_root_path = Path(_config_files[0]).parent

monai/metrics/__init__.py

@@ -19,7 +19,7 @@
 from .fid import FIDMetric, compute_frechet_distance
 from .froc import compute_fp_tp_probs, compute_fp_tp_probs_nd, compute_froc_curve_data, compute_froc_score
 from .generalized_dice import GeneralizedDiceScore, compute_generalized_dice
-from .hausdorff_distance import HausdorffDistanceMetric, compute_hausdorff_distance, compute_percent_hausdorff_distance
+from .hausdorff_distance import HausdorffDistanceMetric, compute_hausdorff_distance
 from .loss_metric import LossMetric
 from .meandice import DiceHelper, DiceMetric, compute_dice
 from .meaniou import MeanIoU, compute_iou

monai/metrics/generalized_dice.py

@@ -37,28 +37,19 @@ class GeneralizedDiceScore(CumulativeIterationMetric):
         reduction: Define mode of reduction to the metrics. Available reduction modes:
             {``"none"``, ``"mean"``, ``"sum"``, ``"mean_batch"``, ``"sum_batch"``,
             ``"mean_channel"``, ``"sum_channel"``}, default to ``"mean"``. if "none", will not do reduction.
+            Default value is changed from `MetricReduction.MEAN_BATCH` to `MetricReduction.MEAN` in v1.5.0.
+            Old versions computed `mean` when `mean_batch` was provided due to bug in reduction.
         weight_type: {``"square"``, ``"simple"``, ``"uniform"``}. Type of function to transform
             ground truth volume into a weight factor. Defaults to ``"square"``.
 
     Raises:
         ValueError: When the `reduction` is not one of MetricReduction enum.
     """
 
-    @deprecated_arg_default(
-        "reduction",
-        old_default=MetricReduction.MEAN_BATCH,
-        new_default=MetricReduction.MEAN,
-        since="1.4.0",
-        replaced="1.5.0",
-        msg_suffix=(
-            "Old versions computed `mean` when `mean_batch` was provided due to bug in reduction， "
-            "If you want to retain the old behavior (calculating the mean), please explicitly set the parameter to 'mean'."
-        ),
-    )
     def __init__(
         self,
         include_background: bool = True,
-        reduction: MetricReduction | str = MetricReduction.MEAN_BATCH,
+        reduction: MetricReduction | str = MetricReduction.MEAN,
         weight_type: Weight | str = Weight.SQUARE,
     ) -> None:
         super().__init__()

monai/metrics/hausdorff_distance.py

@@ -28,7 +28,7 @@
 
 from .metric import CumulativeIterationMetric
 
-__all__ = ["HausdorffDistanceMetric", "compute_hausdorff_distance", "compute_percent_hausdorff_distance"]
+__all__ = ["HausdorffDistanceMetric", "compute_hausdorff_distance"]
 
 
 class HausdorffDistanceMetric(CumulativeIterationMetric):
@@ -216,31 +216,3 @@ def _compute_percentile_hausdorff_distance(
     if 0 <= percentile <= 100:
         return torch.quantile(surface_distance, percentile / 100)
     raise ValueError(f"percentile should be a value between 0 and 100, get {percentile}.")
-
-
-@deprecated(since="1.3.0", removed="1.5.0")
-def compute_percent_hausdorff_distance(
-    edges_pred: np.ndarray,
-    edges_gt: np.ndarray,
-    distance_metric: str = "euclidean",
-    percentile: float | None = None,
-    spacing: int | float | np.ndarray | Sequence[int | float] | None = None,
-) -> float:
-    """
-    This function is used to compute the directed Hausdorff distance.
-    """
-
-    surface_distance: np.ndarray = get_surface_distance(  # type: ignore
-        edges_pred, edges_gt, distance_metric=distance_metric, spacing=spacing
-    )
-
-    # for both pred and gt do not have foreground
-    if surface_distance.shape == (0,):
-        return np.nan
-
-    if not percentile:
-        return surface_distance.max()  # type: ignore[no-any-return]
-
-    if 0 <= percentile <= 100:
-        return np.percentile(surface_distance, percentile)  # type: ignore[no-any-return]
-    raise ValueError(f"percentile should be a value between 0 and 100, get {percentile}.")

monai/metrics/utils.py

@@ -131,9 +131,6 @@ def do_metric_reduction(
     return f, not_nans
 
 
-@deprecated_arg_default(
-    name="always_return_as_numpy", since="1.3.0", replaced="1.5.0", old_default=True, new_default=False
-)
 @deprecated_arg(
     name="always_return_as_numpy",
     since="1.5.0",
@@ -146,7 +143,7 @@ def get_mask_edges(
     label_idx: int = 1,
     crop: bool = True,
     spacing: Sequence | None = None,
-    always_return_as_numpy: bool = True,
+    always_return_as_numpy: bool = False,
 ) -> tuple[NdarrayTensor, NdarrayTensor]:
     """
     Compute edges from binary segmentation masks. This
@@ -175,6 +172,7 @@ def get_mask_edges(
             otherwise `scipy`'s binary erosion is used to calculate the edges.
         always_return_as_numpy: whether to a numpy array regardless of the input type.
             If False, return the same type as inputs.
+            The default value is changed from `True` to `False` in v1.5.0.
     """
     # move in the funciton to avoid using all the GPUs
     cucim_binary_erosion, has_cucim_binary_erosion = optional_import("cucim.skimage.morphology", name="binary_erosion")

monai/networks/nets/swin_unetr.py

@@ -50,16 +50,8 @@ class SwinUNETR(nn.Module):
     <https://arxiv.org/abs/2201.01266>"
     """
 
-    @deprecated_arg(
-        name="img_size",
-        since="1.3",
-        removed="1.5",
-        msg_suffix="The img_size argument is not required anymore and "
-        "checks on the input size are run during forward().",
-    )
     def __init__(
         self,
-        img_size: Sequence[int] | int,
         in_channels: int,
         out_channels: int,
         patch_size: int = 2,
@@ -83,10 +75,6 @@ def __init__(
     ) -> None:
         """
         Args:
-            img_size: spatial dimension of input image.
-                This argument is only used for checking that the input image size is divisible by the patch size.
-                The tensor passed to forward() can have a dynamic shape as long as its spatial dimensions are divisible by 2**5.
-                It will be removed in an upcoming version.
             in_channels: dimension of input channels.
             out_channels: dimension of output channels.
             patch_size: size of the patch token.

monai/transforms/croppad/array.py

@@ -809,13 +809,12 @@ def threshold_at_one(x):
 
     """
 
-    @deprecated_arg_default("allow_smaller", old_default=True, new_default=False, since="1.2", replaced="1.5")
     def __init__(
         self,
         select_fn: Callable = is_positive,
         channel_indices: IndexSelection | None = None,
         margin: Sequence[int] | int = 0,
-        allow_smaller: bool = True,
+        allow_smaller: bool = False,
         return_coords: bool = False,
         k_divisible: Sequence[int] | int = 1,
         mode: str = PytorchPadMode.CONSTANT,
@@ -830,7 +829,8 @@ def __init__(
             margin: add margin value to spatial dims of the bounding box, if only 1 value provided, use it for all dims.
             allow_smaller: when computing box size with `margin`, whether to allow the image edges to be smaller than the
                 final box edges. If `False`, part of a padded output box might be outside of the original image, if `True`,
-                the image edges will be used as the box edges. Default to `True`.
+                the image edges will be used as the box edges. Default to `False`.
+                The default value is changed from `True` to `False` in v1.5.0.
             return_coords: whether return the coordinates of spatial bounding box for foreground.
             k_divisible: make each spatial dimension to be divisible by k, default to 1.
                 if `k_divisible` is an int, the same `k` be applied to all the input spatial dimensions.

monai/transforms/croppad/dictionary.py

@@ -716,15 +716,14 @@ class CropForegroundd(Cropd):
     for more information.
     """
 
-    @deprecated_arg_default("allow_smaller", old_default=True, new_default=False, since="1.2", replaced="1.5")
     def __init__(
         self,
         keys: KeysCollection,
         source_key: str,
         select_fn: Callable = is_positive,
         channel_indices: IndexSelection | None = None,
         margin: Sequence[int] | int = 0,
-        allow_smaller: bool = True,
+        allow_smaller: bool = False,
         k_divisible: Sequence[int] | int = 1,
         mode: SequenceStr = PytorchPadMode.CONSTANT,
         start_coord_key: str | None = "foreground_start_coord",
@@ -744,7 +743,8 @@ def __init__(
             margin: add margin value to spatial dims of the bounding box, if only 1 value provided, use it for all dims.
             allow_smaller: when computing box size with `margin`, whether to allow the image edges to be smaller than the
                 final box edges. If `False`, part of a padded output box might be outside of the original image, if `True`,
-                the image edges will be used as the box edges. Default to `True`.
+                the image edges will be used as the box edges. Default to `False`.
+                The default value is changed from `True` to `False` in v1.5.0.
             k_divisible: make each spatial dimension to be divisible by k, default to 1.
                 if `k_divisible` is an int, the same `k` be applied to all the input spatial dimensions.
             mode: available modes for numpy array:{``"constant"``, ``"edge"``, ``"linear_ramp"``, ``"maximum"``,