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

Added the option to use a variable path by starting the argument with $.... #10

Merged
merged 3 commits into from
Oct 23, 2013

Conversation

mpelko
Copy link
Contributor

@mpelko mpelko commented Oct 23, 2013

... Also the cachedir can be inferred from the path.

This is not tested in a notebook.

… $. Also the cachedir can be inferred from the path.
* When you execute this cell the first time, the code is executed, and the variables `var1` and `var2` are saved in `mycache.pkl` in the current directory along with the outputs. Rich display outputs are only saved if you use the development version of IPython. When you execute this cell again, the code is skipped, the variables are loaded from the file and injected into the namespace, and the outputs are restored in the notebook.


* Alternatively use $file_name instead mycache.pkl, where "file_name" is a variable holding the path to the file used for caching.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of

@rossant
Copy link
Owner

rossant commented Oct 23, 2013

Should be tested in the notebook.

@mpelko mpelko closed this Oct 23, 2013
@mpelko mpelko reopened this Oct 23, 2013
@rossant
Copy link
Owner

rossant commented Oct 23, 2013

Looks good, thanks! Merging.

rossant added a commit that referenced this pull request Oct 23, 2013
Added the option to use a variable path by starting the argument with $....
@rossant rossant merged commit 01369e2 into rossant:master Oct 23, 2013
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.

2 participants