Warn instead of fail unknown inputs

Author: nevillelyh

python/coglet/file_runner.py

@@ -206,6 +206,9 @@ async def _predict(self, pid: str, req: Dict[str, Any]) -> None:
 
             req_in = req['input']
             for k, v in req_in.items():
+                # ignore unknown input fields, let runner check for them and print warning
+                if k not in self.runner.inputs:
+                    continue
                 req_in[k] = self.runner.inputs[k].type.json_decode(v)
 
             if self.runner.is_iter():

python/coglet/inspector.py

@@ -266,9 +266,12 @@ def check_input(
 ) -> Dict[str, Any]:
     kwargs: Dict[str, Any] = {}
     for name, value in inputs.items():
-        assert name in adt_ins, f'unknown input field: {name}'
-        adt_in = adt_ins[name]
-        kwargs[name] = adt_in.type.normalize(value)
+        # assert name in adt_ins, f'unknown input field: {name}'
+        adt_in = adt_ins.get(name)
+        if adt_in is None:
+            print(f'WARNING unknown input field ignored: {name}')
+        else:
+            kwargs[name] = adt_in.type.normalize(value)
     for name, adt_in in adt_ins.items():
         if name not in kwargs:
             # default=None is only allowed on `Optional[<type>]`

python/tests/bad_inputs/simple.py

@@ -16,7 +16,9 @@
         {'i': 0, 's': 'foo'},
         'missing required input field: xs',
     ),
-    ({'x': 0}, 'unknown input field: x'),
+    # Allow unknown inputs for backwards compatibility
+    # Print a warning instead as part of logs
+    # ({'x': 0}, 'unknown input field: x'),
 ]