-
Notifications
You must be signed in to change notification settings - Fork 208
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
th -e "require 'cutorch'" ...s/anthonyyuan/torch/install/share/lua/5.1/trepl/init.lua:389: attempt to index a string value #660
Comments
I get the same error after a torch clean install |
I looked at the last pull requests and found :
it works for me |
i'm checking with a new install |
i'm not able to reproduce it with a fresh torch install. do i have to install it on a specific OS or version? |
I just did this to reproduce the issue.
|
@Cadene can you help me debug this one.
Also, if that fails,
|
|
oh. for some reason, there seems to be a global variable called "require" (i.e. _G.require) that is a string. |
does this happen when loading any other package?
I will try to reproduce this somewhere. |
same with cunn
|
nn, image, rnn, tds, torchnet works |
hmmm. i think any trigger to paths.require is failing.
|
and if that fails too, any chance you can give me ssh to the machine. it will take me much longer to setup a debian. All you will have to do is run a command on your machine to ssh into my server, so that i can get a reverse tunnel. Let's talk details on torch slack |
|
How far does this issue go now? I got similar issue, but a different error message.
|
@ruotianluo what OS? Ubuntu? Debian? |
@soumith CentOS Linux release 7.2.1511 (Core) |
@soumith So what's actually the reason that causes this problem? |
I got to this thread in search for a solution to this very issue. I am getting the same error, after I updating my torch,nn,cunn,cudnn and cutorch libs.
|
this is so frustrating, i am not able to reproduce this issue anywhere. |
I can give you ssh access to my server. What's strange is those commands are running just fine on my personal desktop. |
okay, can you email me at [redacted] we can figure out ssh access details. No it is not CUDA 7.5, i've already tested this. |
@drimpossible I got almost the same situation, but my desktop is also cuda 7.5. |
@soumith Any progress? |
until i get a reproduction, i dont know how to fix it. any public access ssh (so that i can login) to a machine that has this problem will be helpful. |
@soumith Using binary search, I found the error doesn't appear if I roll back all the repositories before 12.28. And the error will occur if roll back to around 12.30. Then I tried to find what exact commit in which package causes the error. It turns out, if I checkout the cutorch to commit 1ac0668, i will get the error. |
thanks for bisecting it. cc: @gchanan something broke on your commit. |
Great! Since I can't reproduce the issue, @ruotianluo can you revert the changes to init.lua and Tensor.lua from that commit separately and tell me if either (or both) fixes the issue? |
@gchanan Reverting either or both don't fix the issue. |
@ruotianluo okay, let me prepare a few other commits for you to try out. Thanks for helping track this down! |
@ruotianluo can you run "nvcc --version" -- what version does it say you are running? |
@ruotianluo can you try the following branches and tell me if any of them work? (they are all single commits off the commit you identified) |
I should point out that these branches are just for testing "require 'cutorch'" -- functionality beyond that is expected to be broken. |
@gchanan torchgenericstorage and genericstorage don't work.(the same error) |
can you try genericstoragetensor with init.Lua and Tensor.Lua rolled back as before? |
It works. |
hmm, I'm still not sure what's going on here -- thanks for your continuing help. Can you try https://github.com/gchanan/cutorch/tree/thchalfh ? (it shouldn't matter what you do with init.lua and tensor.lua) |
This doesn't work. |
None of these works. |
Something very strange is going on...like the symbol generation is getting mixed up between torch and cutorch. Can you try https://github.com/gchanan/cutorch/tree/generateStorageTH? |
Doesn't work either. |
@ruotianluo I sent you an e-mail, it would probably be more productive if we were able to find a time that works for both of us to sit in the torch gitter and debug in real time. In any case, can you do the following? Then try: |
Only genericstoragetensor_genseparate works. |
Here is my It turns out there's another old torch installation on my system. So check if you have any old torch installed on your LUA_PATH, @Cadene @drimpossible ; it could be the same reason. And thank gchanan for his help. |
I tried cleaning the above things and ran into a lot more, so I can't pinpoint the problem precisely but more or less it was old torch installation. Path problems compounded the issue too. It works fine now. The above comment really helped. Thanks @ruotianluo |
@ruotianluo I think I have a similar problem - at some point I installed a version of torch that didn't work. I've tried again and got this far but am getting these errors. However, I'm not sure what my
I can't work out what shouldn't be there... the couple of bits I've tried deleting just result in the same or different errors... |
@philgyford don't change your lua_path path, just delete you other versions. |
@ruotianluo Thanks, but it was a while ago and I don't know exactly what was installed where... |
@philgyford then I guess you need to search through the lua_path to see which directory it's in. Just to make sure, you at least reinstall the latest torch somewhere right? |
@ruotianluo Yes, I recently installed it in |
I get this error: /home/myName/torch/install/bin/lua: /home/myNmae/torch/install/share/lua/5.2/trepl/init.lua:389: /home/myName/torch/install/share/lua/5.2/hdf5/ffi.lua:73: expected align(#) on line 689 |
No description provided.
The text was updated successfully, but these errors were encountered: