_utils._create_file should use contextual deletion about oct2py HOT 7 CLOSED

Sounds reasonable, will do.

from oct2py.

Actually, when I looked closer this is not as easy as it appears. The file name is used by both python and Octave, and I do not see a clean place to use a context. Were you intending to use a context as a user, or was this intended to be an implementation fix?

from oct2py.

You'd have to expose the context to the user: with octave() as oc: ...
On Jun 9, 2013 11:16 PM, "Steven Silvester" [email protected]
wrote:

Actually, when I looked closer this is not as easy as it appears. The file
name is used by both python and Octave, and I do not see a clean place to
use a context. Were you intending to use a context as a user, or was this
intended to be an implementation fix?

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/9#issuecomment-19174949
.

from oct2py.

Right, but that does not solve the problem of the same file needing to be opened independently by Octave and Python. I cannot pass a file handle to Octave, so far as I know.

from oct2py.

You can create a NamedTemporaryFile that is deleted at the end of the
context
On Jun 9, 2013 11:41 PM, "Steven Silvester" [email protected]
wrote:

Right, but that does not solve the problem of the same file needing to be
opened independently by Octave and Python. I cannot pass a file handle to
Octave, so far as I know.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/9#issuecomment-19175403
.

from oct2py.

Okay, sorry I've been away for so long. Oct2py can now be used with context. I could not find a way to use the NamedTemporaryFile context manager, since I have to return control from within the__enter__ function (resulting in the file being deleted). Instead I clear the files upon closing or when the __exit__ function is called.

Example:

    oc = Oct2Py()
    with oc as oc1:
        ones = oc1.ones(1)
    assert ones == np.ones(1)

    with oc as oc2:
        zeros = oc2.zeros(3)
    assert np.allclose(zeros, np.zeros((3, 3)))

from oct2py.

I'm calling this one licked. Thanks again.

from oct2py.