fix update bug in drop

Author: risenW

danfojs/src/core/frame.js

@@ -89,7 +89,7 @@ export class DataFrame extends Ndframe {
                 }
                 return col_idx
             });
-            
+
             this.col_data.forEach((col, idx) => {
                 if (!index.includes(idx)) {
                     new_col_data[self.column_names[idx]] = col
@@ -107,7 +107,7 @@ export class DataFrame extends Ndframe {
             } else {
                 let old_cols = self.columns
                 let new_columns = Object.keys(new_col_data)
-                this.__update_frame_in_place(null, null, new_col_data, null,new_dtype)
+                this.__update_frame_in_place(null, null, new_col_data, null, new_dtype)
                 this.__set_col_property(self, self.col_data, new_columns, old_cols)
 
             }
@@ -696,7 +696,8 @@ export class DataFrame extends Ndframe {
         let df = new DataFrame([...this.values],
             {
                 columns: [...this.column_names],
-                index: this.index, dtypes: this.dtypes
+                index: this.index,
+                dtypes: this.dtypes
             })
         return df
     }
@@ -717,7 +718,6 @@ export class DataFrame extends Ndframe {
     }
 
     /**
-    * Generate a new DataFrame with the specified index.
     * Set the DataFrame index (row labels) using an array of the same length.
     * @param {kwargs} {index: Array of new index values}
     */
@@ -861,9 +861,8 @@ export class DataFrame extends Ndframe {
             })
 
             if (utils.__key_in_object(kwargs, "inplace") && kwargs['inplace'] == true) {
-                this.data = new_row_data
-                this.index_arr = sorted_index
-                return null
+                this.__update_frame_in_place(new_row_data, null, null, sorted_index, null)
+
             } else {
                 let df = new DataFrame(new_row_data, { columns: this.column_names, index: sorted_index, dtype: this.dtypes })
                 return df
@@ -1094,17 +1093,14 @@ export class DataFrame extends Ndframe {
     groupby(col) {
 
         let len = this.shape[0]
-
         let column_names = this.column_names
         let col_dict = {};
         let key_column = null;
 
         if (col.length == 2) {
-
             if (column_names.includes(col[0])) {
                 // eslint-disable-next-line no-unused-vars
                 var [data1, col_name1] = indexLoc(this, { "rows": [`0:${len}`], "columns": [`${col[0]}`], "type": "loc" });
-
             }
             else {
                 throw new Error(`column ${col[0]} does not exist`);
@@ -1307,37 +1303,6 @@ export class DataFrame extends Ndframe {
     }
 
 
-
-    // let new_row_data = []
-    // let row_data = this.values;
-    // let columns = this.column_names;
-
-    // row_data.map(arr=>{
-    //     let temp_arr = []
-    //     arr.map(val=>{
-    //         if (isNaN(val) && typeof val != "string" ){
-    //             temp_arr.push(true)
-    //         }else{
-    //             temp_arr.push(false)
-    //         }
-    //     })
-    //     new_row_data.push(temp_arr)
-    // })
-
-    // // for (let i = 0; i < values.length; i++) {
-    // //     let temp_data = []
-    // //     let row_value = values[i]
-    // //     for (let j = 0; j < row_value.length; j++) {
-
-    // //         let val = row_value[j] == 0 ? true : !row_value[j]
-    // //         temp_data.push(val)
-    // //     }
-    // //     data.push(temp_data);
-    // // }
-
-    // return new DataFrame(new_row_data, { columns: columns, index: this.index })
-
-
     /**
      * Obtain index containing nan values
      * @return Array list (int)