Skip to content

Abstract tabular dataset loading#274

Open
mttk wants to merge 10 commits intomasterfrom
abstract_tabular_loading
Open

Abstract tabular dataset loading#274
mttk wants to merge 10 commits intomasterfrom
abstract_tabular_loading

Conversation

@mttk
Copy link
Member

@mttk mttk commented Jan 14, 2021

Closes #273

Draft, function calls and names subject to change, but the gist is here.

@mttk mttk self-assigned this Jan 14, 2021
Copy link
Collaborator

@ivansmokovic ivansmokovic left a comment

Choose a reason for hiding this comment

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

Good work. Some nitpicks.
Not an actual review, just wanted to point some things out.

@mttk mttk requested a review from ivansmokovic January 16, 2021 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove from_tabular_file in ArrowDataset

2 participants