Obey _recursive_ parameter on non-target nodes

hydra/_internal/instantiate/_instantiate2.py

@@ -354,18 +354,24 @@ def instantiate_node(
             ):
                 dict_items = {}
                 for key, value in node.items():
-                    # list items inherits recursive flag from the containing dict.
-                    dict_items[key] = instantiate_node(
-                        value, convert=convert, recursive=recursive
-                    )
+                    if recursive:
+                        # list items inherits recursive flag from the containing dict.
+                        dict_items[key] = instantiate_node(
+                            value, convert=convert, recursive=recursive
+                        )
+                    else:
+                        dict_items[key] = value
                 return dict_items
             else:
                 # Otherwise use DictConfig and resolve interpolations lazily.
                 cfg = OmegaConf.create({}, flags={"allow_objects": True})
                 for key, value in node.items():
-                    cfg[key] = instantiate_node(
-                        value, convert=convert, recursive=recursive
-                    )
+                    if recursive:
+                        cfg[key] = instantiate_node(
+                            value, convert=convert, recursive=recursive
+                        )
+                    else:
+                        cfg[key] = value
                 cfg._set_parent(node)
                 cfg._metadata.object_type = node._metadata.object_type
                 if convert == ConvertMode.OBJECT:

tests/instantiate/__init__.py

@@ -420,6 +420,11 @@ class NestedConf:
     b: Any = field(default_factory=lambda: User(name="b", age=2))
 
 
+@dataclass
+class NestedConfNoTarget:
+    a: Any = field(default_factory=lambda: SimpleClassDefaultPrimitiveConf)
+
+
 def recisinstance(got: Any, expected: Any) -> bool:
     """Compare got with expected type, recursively on dict and list."""
     if not isinstance(got, type(expected)):

tests/instantiate/test_instantiate.py

@@ -33,6 +33,7 @@
     Mapping,
     MappingConf,
     NestedConf,
+    NestedConfNoTarget,
     NestingClass,
     OuterClass,
     Parameters,
@@ -2096,3 +2097,13 @@ class DictValuesConf:
     cfg = OmegaConf.structured(DictValuesConf)
     obj = instantiate_func(config=cfg)
     assert obj.d is None
+
+
+
+def test_non_target_recursive(instantiate_func: Any) -> None:
+    cfg = OmegaConf.structured(NestedConfNoTarget)
+    obj = instantiate_func(config=cfg, _recursive_=False)
+    assert isinstance(obj.a, DictConfig)
+
+    obj = instantiate_func(config=cfg, _recursive_=True)
+    assert isinstance(obj.a, SimpleClass)