Skip to content

Conversation

@DEKHTIARJonathan
Copy link
Member

@DEKHTIARJonathan DEKHTIARJonathan commented Apr 17, 2018

Quick fix to removed console being bombarded by the download of a dataset (e.g. MNIST).

Can be tested like this:

import tensorlayer as tl
X_train, y_train, X_val, y_val, X_test, y_test = tl.files.load_mnist_dataset()

Be careful to reinstall the library properly if you want to test before the next release

pip install -r requirements.txt
pip install -e .[test]

Result

tensorlayer

@zsdonghao
Copy link
Member

it that good to include a library for this task only? @lgarithm what do you think?

@lgarithm
Copy link
Member

lgarithm commented Apr 17, 2018 via email

@DEKHTIARJonathan
Copy link
Member Author

@zsdonghao if you prefer I can vendor the package. However, this will prevent us from getting the package updates.

As @lgarithm pointed out it is a small lib which does one thing, and do it very well. It could also be used for tl.train API

@zsdonghao zsdonghao merged commit 8ded440 into master Apr 17, 2018
@DEKHTIARJonathan DEKHTIARJonathan deleted the ProgressBar-DownloadDatasetFix branch April 18, 2018 12:38
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.

4 participants