Skip to content
This repository has been archived by the owner on Jan 9, 2024. It is now read-only.

Commit

Permalink
Fix issue #46
Browse files Browse the repository at this point in the history 
1. Fixed the argument passed to Intent.column_summary from a series to a single column DataFrame.
2. Change the data type of some summary statistics to float from int64 so that they are json serializable.
  • Loading branch information
jichao committed Mar 6, 2019
1 parent 756c12e commit e920388
Show file tree
Hide file tree
Showing 4 changed files with 48,886 additions and 8 deletions.

2 comments on commit e920388

@jichaoz
Copy link
Contributor

@jichaoz jichaoz commented on e920388 Mar 7, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @adithyabsk, Can you help review this code? I was supposed to push this change to a new branch and create a pull request, but accidentally pushed to the development branch. Since this is a relatively straight-forward commit, if you provides your comment, I can address in a follow-up commit. Thanks.

@adithyabsk
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@jichaogp LGTM

Please sign in to comment.