Skip to content
This repository has been archived by the owner on May 30, 2019. It is now read-only.

change API for dataset #470

Open
ry opened this issue Mar 30, 2018 · 0 comments
Open

change API for dataset #470

ry opened this issue Mar 30, 2018 · 0 comments

Comments

@ry
Copy link
Contributor

ry commented Mar 30, 2018

Currently the dataset is specified as a string:

pr.dataset("cifar10/train")
pr.dataset("iris")

It would be better to use actual functions for each. example:

pr.dataset.cifar10("train")
pr.dataset.iris()

This allows for type checking on the components of the datasets - that is cifar10 has { images, labels } and iris has { features, labels } - thus the docs will better reflect the information people need.

splitName should default to "train"

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant