Merge pull request #134 from worc3131/kwdefaults

Fix copy_func to include required keyword defaults

Author: jph00

fastcore/foundation.py

@@ -22,6 +22,7 @@ def copy_func(f):
     "Copy a non-builtin function (NB `copy.copy` does not work for this)"
     if not isinstance(f,FunctionType): return copy(f)
     fn = FunctionType(f.__code__, f.__globals__, f.__name__, f.__defaults__, f.__closure__)
+    fn.__kwdefaults__ = f.__kwdefaults__
     fn.__dict__.update(f.__dict__)
     return fn
 

nbs/01_foundation.ipynb

@@ -73,6 +73,7 @@
     "    \"Copy a non-builtin function (NB `copy.copy` does not work for this)\"\n",
     "    if not isinstance(f,FunctionType): return copy(f)\n",
     "    fn = FunctionType(f.__code__, f.__globals__, f.__name__, f.__defaults__, f.__closure__)\n",
+    "    fn.__kwdefaults__ = f.__kwdefaults__\n",
     "    fn.__dict__.update(f.__dict__)\n",
     "    return fn"
    ]
@@ -118,6 +119,17 @@
     "assert c is not foo"
    ]
   },
+  {
+   "cell_type": "code",
+   "execution_count": null,
+   "metadata": {},
+   "outputs": [],
+   "source": [
+    "def g(x, *, y=3):\n",
+    "    return x+y\n",
+    "test_eq(copy_func(g)(4), 7)"
+   ]
+  },
   {
    "cell_type": "code",
    "execution_count": null,