fix: review cmt - error when tp_plan not set for tp_size

Signed-off-by: Mehant Kammakomati <[email protected]>

Author: kmehant

src/transformers/modeling_utils.py

@@ -3961,7 +3961,8 @@ def from_pretrained(
             raise ValueError(
                 "`state_dict` cannot be passed together with a model name or a `gguf_file`. Use one of the two loading strategies."
             )
-
+        if tp_size is not None and tp_plan is None:
+            raise ValueError(f"tp_plan has to be set when tp_size is passed.")
         if tp_plan is not None and tp_plan != "auto":
             # TODO: we can relax this check when we support taking tp_plan from a json file, for example.
             raise ValueError(f"tp_plan supports 'auto' only for now but got {tp_plan}.")