Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork175
Add yielding support to mlua#518
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
base:main
Are you sure you want to change the base?
Conversation
8ce9826
to3aaf510
Comparewoops |
cheesycod commentedJan 26, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Oh wow, ig someone managed to do the impossible . Still seems to be some errors left to fix though like ``Assertion failed: (((nresults) < (L->top - L->ci->func)) && "not enough elements in the stack"), function lua_yieldk, file ldo.c, line 697. |
It still has many many issues, as i have to modify callback_error_ext |
cheesycod commentedJan 26, 2025
How exactly does this solve the issue of the Rust/Lua call boundary? |
This doesn't, it relies on I'm planning to add both this and an async system where Lua acts as the waker. |
cheesycod commentedMay 3, 2025
Any updates on this PR@radiantgurl |
cheesycod commentedJun 5, 2025
@radiantgurl@khvzak Have integrated this PR into my general PR for yielding and luau continuations (which go together anyways as you cant use continuations really without the yielding):#588 |
Ok, can you co-author me on the specific commit that integrates this PR? Sorry that i was basically dead, i have a bunch of volunteering activities irl. |
cheesycod commentedJun 5, 2025 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
How do you co author someone on a commit (im new to this type of git stuff) Also, I didn’t really use much of your code in the end besides the inspiration for yielded_values in extra_args and the is_yieldable bit. The rest had to be reworked a lot (especially to work with luau specific stuff in its continuation implementation) and due to general errors I found when testing/trying to use it (turns out yielding isn’t as easy as it looks lol) Sorry for not attributing you in advance though |
No description provided.