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

Free disk space #255

Merged
merged 3 commits into from Apr 2, 2019
Merged

Free disk space #255

merged 3 commits into from Apr 2, 2019

Conversation

us
Copy link
Contributor

@us us commented Mar 14, 2019

TODO(tfds): Should try to estimate the available free disk space (if possible) and raise an error if not.
#172 cleaned commit.

TODO(tfds): Should try to estimate the available free disk space (if possible) and raise an error if not.
@googlebot googlebot added the cla: yes Author has signed CLA label Mar 14, 2019
@us us mentioned this pull request Mar 14, 2019
@us
Copy link
Contributor Author

us commented Mar 15, 2019

@rsepassi I got the pyutil error on linux machines, linux need some dependencies same error and solution
Should I add my own function to utils? If psutil fix this, we'll use that again

@Conchylicultor
Copy link
Member

I've added py3.6-dev recently to the open-source installation process.
Hopefully this should unlock you.
https://github.com/tensorflow/datasets/blob/master/oss_scripts/oss_pip_install.sh#L25

You can try to sync and re-run the tests.

@us
Copy link
Contributor Author

us commented Mar 26, 2019

Okay now it's passed.

@rsepassi rsepassi changed the title Free disk space (todo) Free disk space Apr 1, 2019
@rsepassi
Copy link
Contributor

rsepassi commented Apr 1, 2019

Thanks! Will merge shortly.

@tfds-copybara tfds-copybara merged commit 554a2b3 into tensorflow:master Apr 2, 2019
tfds-copybara pushed a commit that referenced this pull request Apr 2, 2019
PiperOrigin-RevId: 241555248
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Author has signed CLA
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants