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

Module 'ne-conj' not found Lua error #172

Open
xxyzz opened this issue Dec 22, 2023 · 2 comments
Open

Module 'ne-conj' not found Lua error #172

xxyzz opened this issue Dec 22, 2023 · 2 comments

Comments

@xxyzz
Copy link
Collaborator

xxyzz commented Dec 22, 2023

Page: https://en.wiktionary.org/wiki/वाक्नु
Error: https://kaikki.org/dictionary/errors/details-Traceback--most-recent-call-last-----F-Plr1Wuzg.html

वाक्नु (Nepali verb) LUA error in #invoke('ne-conj', 'show') parent ('Template:ne-conj', {'i': 'y'})

Traceback (most recent call last):
  File "/home/ubuntu/temp-wiktionary/venv/lib/python3.10/site-packages/wikitextprocessor/luaexec.py", line 745, in call_lua_sandbox
    ret: tuple[bool, str] = ctx.lua_invoke(
  File "lupa/lua51.pyx", line 896, in lupa.lua51._LuaObject.__call__
  File "lupa/lua51.pyx", line 1795, in lupa.lua51.call_lua
  File "lupa/lua51.pyx", line 1821, in lupa.lua51.execute_lua_call
  File "lupa/lua51.pyx", line 1703, in lupa.lua51.raise_lua_error
lupa.lua51.LuaError: [string "_sandbox_phase2"]:179: Could not find module ne-conj: module 'ne-conj' not found
stack traceback:
	[string "_sandbox_phase2"]:179: in function <[string "_sandbox_phase2"]:121>
	[C]: in function 'error'

Unreproducible error, have no idea.

@kristian-clausal
Copy link
Collaborator

Yeah, I bet this won't show up again... I don't think we can do much about these kinds of errors. I'll ask Tatu if he wants to continue hunting for the "LAMBDA" bugs still, but now that you implemented lua invocation isolation it's not as pressing anymore.

I think we might just have to live with random weird memory errors, now that they are isolated and have a smaller impact like this (if this is the same kind of issue, of course). If there is need, we can just run things twice (or thrice) and join together the good parts of the runs into one and discard "bad" parts with errors.

@xxyzz
Copy link
Collaborator Author

xxyzz commented Dec 22, 2023

This exact error happens before and I suspect it might be a wrong cache page from another namespace: c151d39

but that doesn't work...

I think this error could only happen when Lua_loader() returns None and it is different than the lambda arg error.

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

No branches or pull requests

2 participants