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

Use symbols rather than modules in ispoisoned #7

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Use symbols rather than modules in ispoisoned #7

wants to merge 1 commit into from

Conversation

timholy
Copy link

@timholy timholy commented Sep 30, 2014

This allows users to poison packages before they are loaded, forcing manual regeneration of the cached files. This enables the syntax in the README in #3.

@vtjnash
Copy link
Owner

vtjnash commented Oct 1, 2014

this cache needs to be able to track the fullname of a module. it is also preferably able to "unpoison" the module after it has been defined once. I agree that this is needed – just not certain of the details yet. I think if you could switch it to using fullname, I'll go ahead and merge, since this entire enterprise is really a WIP anyways

This allows users to poison packages before they are loaded,
forcing manual regeneration of the cached files.
@timholy
Copy link
Author

timholy commented Oct 1, 2014

OK, this uses the fullname.

As an alternative to unpoisoning, perhaps the better solution is to maintain a separate cache for "poison-once" packages. Speed.regenerate!(:Images)? As it is, at least now users can start julia, say Speed.poison(:Images); using Images, quit julia & restart, and then they have what they want.

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