Skip to content

Commit

Permalink
Merge pull request #441 from ankitskvmdam/patch-1
Browse files Browse the repository at this point in the history
fix: fillNa throws an error when values is an empty string
  • Loading branch information
risenW committed Apr 15, 2022
2 parents 2c274ee + 9b20ffa commit 0f85cae
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/danfojs-base/core/frame.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2102,7 +2102,7 @@ export default class DataFrame extends NDframe implements DataFrameInterface {
): DataFrame | void {
let { columns, inplace } = { inplace: false, ...options }

if (!values && typeof values !== "boolean" && typeof values !== "number") {
if (!values && typeof values !== "boolean" && typeof values !== "number" && typeof values !== "string") {
throw Error('ParamError: value must be specified');
}

Expand Down

0 comments on commit 0f85cae

Please sign in to comment.