deepchem / deepchem/deepchem

NamedTemporaryFile function under windows

Open
#707 10 comments 2 reactions 0 assignees View on GitHub
Contribution Welcome Good First Contribution
Dominant language
Python
Stars
7k
Forks
2.3k
PR merge metrics
No merged PRs in 30d

Description

Hi,
I tried to run the MUV inotebook and got some errors with
> [Errno 13] Permission denied

when trying to read or write from temp file.

I found the error online with the solution
[http://zachmoshe.com/2017/04/03/pickling-keras-models.html](url)
And adding delete=False to the 4 instances of NamedTemporaryFile in the file deepchem\hyper\__init__.py from
`tempfile.NamedTemporaryFile()` to
`tempfile.NamedTemporaryFile(delete=False)`
solved the problem.

However I don't know if this option will break the module to run on Linux, and also if this is desirable (default is to delete the file)
See [https://docs.python.org/3.5/library/tempfile.html#tempfile.NamedTemporaryFile](url) for the doc on NamedTemporaryFile and the windows/Linux behaviour

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.