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

Update DataLoaderRaw.lua #45

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

Update DataLoaderRaw.lua #45

wants to merge 1 commit into from

Conversation

kmader
Copy link

@kmader kmader commented Dec 12, 2015

the path is required from pl.paths otherwise line 46 does not work (for using in other scripts)

the path is required from pl.paths otherwise line 46 does not work (for using in other scripts)
@karpathy
Copy link
Owner

Why did this work for me on my machine without the line?

@soumith
Copy link

soumith commented Dec 12, 2015

pl.path is loaded by default in "th", maybe @kmader is trying to use the luajit or lua binary directly.

@kmader
Copy link
Author

kmader commented Dec 13, 2015

@soumith @karpathy I am using it inside of iTorch (https://github.com/facebook/iTorch) which evidently does not load or some how the namespace is being cleared.

@jhorrell
Copy link

jhorrell commented Jan 8, 2016

Not sure, but this issue might be related to #40

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.

4 participants