Fixed failure when setting individual dict key values for subclasses and .init_args. is included in argument #251.

Author: mauvilsa

CHANGELOG.rst

@@ -24,6 +24,9 @@ Fixed
   contexts `#247 <https://github.com/omni-us/jsonargparse/issues/247>`__.
 - Failure with dataclasses that have field with ``init=False`` `#252
   <https://github.com/omni-us/jsonargparse/issues/252>`__.
+- Failure when setting individual dict key values for subclasses and
+  ``.init_args.`` is included in argument `#251
+  <https://github.com/omni-us/jsonargparse/issues/251>`__.
 
 
 v4.20.0 (2023-02-20)

jsonargparse/typehints.py

@@ -394,7 +394,10 @@ def __call__(self, *args, **kwargs):
         cfg, val, opt_str = args[1:]
         if not (self.nargs == '?' and val is None):
             if isinstance(opt_str, str) and opt_str.startswith(f'--{self.dest}.'):
-                sub_opt = opt_str[len(f'--{self.dest}.'):]
+                if opt_str.startswith(f'--{self.dest}.init_args.'):
+                    sub_opt = opt_str[len(f'--{self.dest}.init_args.'):]
+                else:
+                    sub_opt = opt_str[len(f'--{self.dest}.'):]
                 val = NestedArg(key=sub_opt, val=val)
             append = opt_str == f'--{self.dest}+'
             val = self._check_type(val, append=append, cfg=cfg)
@@ -806,11 +809,7 @@ def subclass_spec_as_namespace(val, prev_val=None):
         if '.' not in key:
             root_key = key
         else:
-            if key.startswith('init_args.'):
-                root_key = 'init_args'
-                key = key[len('init_args.'):]
-                val = Namespace({key: val})
-            elif key.startswith('dict_kwargs.'):
+            if key.startswith('dict_kwargs.'):
                 root_key = 'dict_kwargs'
                 key = key[len('dict_kwargs.'):]
                 val = {key: val}

jsonargparse_tests/test_typehints.py

@@ -230,6 +230,25 @@ def test_dict_items(self):
         self.assertEqual(cfg.dict, {'one': 1, 'two': 2})
 
 
+    def test_subclass_dict_items(self):
+        class Class:
+            def __init__(self, param: Dict[str, int]):
+                pass
+
+        parser = ArgumentParser(exit_on_error=False)
+        parser.add_argument('--val', type=Class)
+
+        with mock_module(Class) as module:
+            for init_args in ['', '.init_args']:
+                cfg = parser.parse_args([
+                    f'--val={module}.Class',
+                    f'--val{init_args}.param.one=1',
+                    f'--val{init_args}.param.two=2',
+                ])
+                self.assertEqual(cfg.val.class_path, f'{module}.Class')
+                self.assertEqual(cfg.val.init_args.param, {'one': 1, 'two': 2})
+
+
     def test_dict_union(self):
         class MyEnum(Enum):
             ab = 1