Revised function call.

Moved __get_num_digits() function call into the radix() function, so the only parameter when calling radix() is a List.

Author: joeyajames

Sorting Algorithms/Radix_Sort.ipynb

@@ -10,7 +10,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 5,
+   "execution_count": 1,
    "metadata": {},
    "outputs": [],
    "source": [
@@ -24,7 +24,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 6,
+   "execution_count": 2,
    "metadata": {},
    "outputs": [],
    "source": [
@@ -34,13 +34,22 @@
     "    return reduce(lambda x, y: x + y, A)"
    ]
   },
+  {
+   "cell_type": "markdown",
+   "metadata": {},
+   "source": [
+    "### Changed from YouTube video:\n",
+    "It's much cleaner to put the _get_num_digits call inside the radix function rather than in main as shown in the video. That way you only need to pass a List to the radix function. Thanks to Brother Lui for this suggestion."
+   ]
+  },
   {
    "cell_type": "code",
-   "execution_count": 7,
+   "execution_count": 3,
    "metadata": {},
    "outputs": [],
    "source": [
-    "def radix(A, num_digits):\n",
+    "def radix(A):\n",
+    "    num_digits = __get_num_digits(A)\n",
     "    for digit in range(0, num_digits):\n",
     "        B = [[] for i in range(10)]\n",
     "        for item in A:\n",
@@ -53,7 +62,7 @@
   },
   {
    "cell_type": "code",
-   "execution_count": 9,
+   "execution_count": 4,
    "metadata": {},
    "outputs": [
     {
@@ -68,14 +77,13 @@
    "source": [
     "def main():\n",
     "    A = [55, 45, 3, 289, 213, 1, 288, 53, 2]\n",
-    "    num_digits = __get_num_digits(A)\n",
-    "    A = radix(A, num_digits)\n",
+    "    A = radix(A)\n",
     "    print(A)\n",
     "    \n",
     "    B  = [i for i in range(1000000)]\n",
     "    from random import shuffle\n",
     "    shuffle(B)\n",
-    "    B = radix(B, __get_num_digits(B))\n",
+    "    B = radix(B)\n",
     "    print(B[:6], B[-6:])\n",
     "\n",
     "main()"