docs: Fix oracle doc loader format issue (langchain-ai#19628)

billytrend-cohere · Mar 27, 2024 · 1c27de6 · 1c27de6
1 parent ad77fa1
commit 1c27de6
Showing 1 changed file with 19 additions and 8 deletions.
diff --git a/docs/docs/integrations/document_loaders/oracleadb_loader.ipynb b/docs/docs/integrations/document_loaders/oracleadb_loader.ipynb
@@ -5,18 +5,29 @@
    "source": [
     "# Oracle Autonomous Database\n",
     "\n",
-    "This notebook covers how to load documents from oracle autonomous database, the loader supports connection with connection string or tns config.\n",
+    "Oracle autonomous database is a cloud database that uses machine learning to automate database tuning, security, backups, updates, and other routine management tasks traditionally performed by DBAs.\n",
+    "\n",
+    "This notebook covers how to load documents from oracle autonomous database, the loader supports connection with connection string or tns configuration.\n",
     "\n",
     "## Prerequisites\n",
-    "1. Database runs in a 'Thin' mode.\n",
+    "1. Database runs in a 'Thin' mode:\n",
     "   https://python-oracledb.readthedocs.io/en/latest/user_guide/appendix_b.html\n",
-    "2. `pip install oracledb`\n",
+    "2. `pip install oracledb`:\n",
     "   https://python-oracledb.readthedocs.io/en/latest/user_guide/installation.html"
    ],
    "metadata": {
     "collapsed": false
    }
   },
+  {
+   "cell_type": "markdown",
+   "source": [
+    "## Instructions"
+   ],
+   "metadata": {
+    "collapsed": false
+   }
+  },
   {
    "cell_type": "code",
    "execution_count": null,
@@ -49,7 +60,7 @@
   {
    "cell_type": "markdown",
    "source": [
-    "With mutual TLS authentication (mTLS), wallet_location and wallet_password are required to create the connection, user can create connection by providing either connection string or tns config details."
+    "With mutual TLS authentication (mTLS), wallet_location and wallet_password are required to create the connection, user can create connection by providing either connection string or tns configuration details."
    ],
    "metadata": {
     "collapsed": false
@@ -63,25 +74,25 @@
     "SQL_QUERY = \"select prod_id, time_id from sh.costs fetch first 5 rows only\"\n",
     "\n",
     "doc_loader_1 = OracleAutonomousDatabaseLoader(\n",
+    "    query=SQL_QUERY,\n",
     "    user=s.USERNAME,\n",
     "    password=s.PASSWORD,\n",
     "    schema=s.SCHEMA,\n",
     "    config_dir=s.CONFIG_DIR,\n",
     "    wallet_location=s.WALLET_LOCATION,\n",
     "    wallet_password=s.PASSWORD,\n",
     "    tns_name=s.TNS_NAME,\n",
-    "    query=SQL_QUERY,\n",
     ")\n",
     "doc_1 = doc_loader_1.load()\n",
     "\n",
     "doc_loader_2 = OracleAutonomousDatabaseLoader(\n",
+    "    query=SQL_QUERY,\n",
     "    user=s.USERNAME,\n",
     "    password=s.PASSWORD,\n",
     "    schema=s.SCHEMA,\n",
     "    connection_string=s.CONNECTION_STRING,\n",
     "    wallet_location=s.WALLET_LOCATION,\n",
     "    wallet_password=s.PASSWORD,\n",
-    "    query=SQL_QUERY,\n",
     ")\n",
     "doc_2 = doc_loader_2.load()"
    ],
@@ -107,21 +118,21 @@
    "outputs": [],
    "source": [
     "doc_loader_3 = OracleAutonomousDatabaseLoader(\n",
+    "    query=SQL_QUERY,\n",
     "    user=s.USERNAME,\n",
     "    password=s.PASSWORD,\n",
     "    schema=s.SCHEMA,\n",
     "    config_dir=s.CONFIG_DIR,\n",
     "    tns_name=s.TNS_NAME,\n",
-    "    query=SQL_QUERY,\n",
     ")\n",
     "doc_3 = doc_loader_3.load()\n",
     "\n",
     "doc_loader_4 = OracleAutonomousDatabaseLoader(\n",
+    "    query=SQL_QUERY,\n",
     "    user=s.USERNAME,\n",
     "    password=s.PASSWORD,\n",
     "    schema=s.SCHEMA,\n",
     "    connection_string=s.CONNECTION_STRING,\n",
-    "    query=SQL_QUERY,\n",
     ")\n",
     "doc_4 = doc_loader_4.load()"
    ],