Archipelago icon indicating copy to clipboard operation
Archipelago copied to clipboard

Published 20 hours ago • verified publisher icon ArchipelagoMW

LADX: Full Entrance Rando

Open zig-for opened this issue 1 year ago • 3 comments

What is this fixing or adding?

Adds full ER to LADX. There's a few places where code isn't as efficient as I'd like, but I don't think it will impact perf on asyncs or anything (though open to someone testing that!)

How was this tested?

Multiple players have tested

If this makes graphical changes, please attach screenshots.

No

zig-for avatar Mar 27 '23 21:03 zig-for

Closing for now, but will be back

zig-for avatar Jul 29 '23 02:07 zig-for

We played a dozen of sessions with this merged and tested pretty much all ER options. Everything seems fine 👍. No logic errors encountered so far.

Cybrou avatar Sep 30 '23 01:09 Cybrou

Bump on this, just played a pretty 17 player LADX only async where half of the players or so had this on, no issues

zig-for avatar Feb 14 '24 05:02 zig-for

Zig is no longer actively maintaining LADX.

I'm still willing to merge LADX PRs, but only with a larger volume of peer reviews.

Also, someone needs to sort out the conflicts, obviously

NewSoupVi avatar Jun 21 '24 16:06 NewSoupVi

Unless Zig wants to continue with this PR, someone else would need to reopen it in their stead (with permission of course)

ScipioWright avatar Jun 21 '24 16:06 ScipioWright

Zig gave me permission to take this PR and make a more merge-ready version. I'll close this after I've put up the new PR

ScipioWright avatar Jun 22 '24 17:06 ScipioWright