Skip to content

Commit f29bac5

Browse files
committed
Ignore invalid post-MIP profile values
Signed-off-by: Johannes Rausch <jrausch@nvidia.com>
1 parent a1655ab commit f29bac5

2 files changed

Lines changed: 11 additions & 1 deletion

File tree

examples/puzzletron/run_post_mip_node.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,9 @@ def _register_evaluation_profiles(config: dict) -> None:
4242
continue
4343
node_config = node.get("config")
4444
if isinstance(node_config, Mapping):
45-
profiles.add(node_config.get("profile"))
45+
profile = node_config.get("profile")
46+
if isinstance(profile, str):
47+
profiles.add(profile)
4648
if "qwen35_vlm_realworldqa" in profiles:
4749
from examples.puzzletron.evaluation.vlm.post_mip import register_profiles
4850

tests/unit/torch/puzzletron/test_post_mip_runner.py

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -106,6 +106,14 @@ def test_worker_entrypoint_registers_configured_vlm_evaluation_profile(monkeypat
106106
"type": "downstream_evaluation",
107107
"config": [],
108108
},
109+
"invalid_list_profile": {
110+
"type": "downstream_evaluation",
111+
"config": {"profile": []},
112+
},
113+
"invalid_mapping_profile": {
114+
"type": "downstream_evaluation",
115+
"config": {"profile": {}},
116+
},
109117
"checkpoint_eval": {
110118
"type": "downstream_evaluation",
111119
"config": {"profile": "qwen35_vlm_realworldqa"},

0 commit comments

Comments
 (0)