Merge branch 'lyhue1991-dev' into dev

Author: Rising Odegua

src/danfojs-base/io/node/io.excel.ts

@@ -71,7 +71,7 @@ const $readExcel = async (filePath: string, options: ExcelInputOptionsNode = {})
                     const arrBufInt8 = new Uint8Array(arrBuf);
                     const workbook = read(arrBufInt8, { type: "array", ...parsingOptions });
                     const worksheet = workbook.Sheets[workbook.SheetNames[sheet]];
-                    const data = utils.sheet_to_json(worksheet);
+                    const data = utils.sheet_to_json(worksheet, { defval: null });
                     const df = new DataFrame(data, frameConfig);
                     resolve(df);
                 });
@@ -89,7 +89,7 @@ const $readExcel = async (filePath: string, options: ExcelInputOptionsNode = {})
 
                 const workbook = readFile(filePath, parsingOptions);
                 const worksheet = workbook.Sheets[workbook.SheetNames[sheet]];
-                const data = utils.sheet_to_json(worksheet);
+                const data = utils.sheet_to_json(worksheet, { defval: null });
                 const df = new DataFrame(data, frameConfig);
                 resolve(df);
             })

src/danfojs-node/test/io/excel.reader.test.ts

@@ -106,4 +106,23 @@ describe("toExcel", function () {
         assert.equal(fs.existsSync(filePath), true)
     });
 
+    it("handles null values correctly", async function () {
+        const data = [
+            { 'name': 'Alice', 'age': 25 },
+            { 'name': null, 'age': 30 },
+            { 'name': 'Charlie', 'age': 35 }
+        ];
+        const df: any = new DataFrame(data);
+        const filePath = path.join(process.cwd(), "test", "samples", "people.xlsx");
+        
+        toExcel(df, { filePath, sheetName: 'Sheet1' });
+        const df2: any = await readExcel(filePath);
+        
+        assert.deepEqual(df2.values, [
+            ['Alice', 25],
+            [null, 30],
+            ['Charlie', 35]
+        ]);
+        assert.deepEqual(df2.columns, ['name', 'age']);
+    });
 })