Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added support for DataFrame init with missing values #100

Merged
merged 3 commits into from Jun 21, 2019

Conversation

@AtharvaKhare
Copy link
Contributor

commented Jun 6, 2019

nil is inserted ito represent missing values.

Fixes #21.

AtharvaKhare and others added some commits Jun 6, 2019

Added support for DataFrame init with missing values
`nil` is inserted  ito represent missing values.
Fixed empty select and entire reject for DataFrame
Added tests for:
- creating empty dataframe using `withRows` or `withColumns` variants
- `select:` empty dataframe
- `reject:` entire dataframe
@AtharvaKhare

This comment has been minimized.

Copy link
Contributor Author

commented Jun 19, 2019

This PR should solve #92 and #91.

@AtharvaKhare AtharvaKhare requested a review from olekscode Jun 19, 2019

@AtharvaKhare AtharvaKhare referenced this pull request Jun 20, 2019

@olekscode olekscode merged commit c9088dd into PolyMathOrg:master Jun 21, 2019

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.1%) to 90.446%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.