search

VedalAI / neuro-amongus

Among Us Plugin for Neuro-sama
GNU General Public License v3.0
531 stars 50 forks source link

Minigame solvers #37

Closed ScrubN closed 1 year ago

ScrubN commented 1 year ago

I got bored of doing locomotion so I wrote some solvers.

Currently implemented:

ScrubN commented 1 year ago

@Alexejhero is it possible for solvers to patch into the minigame they are meant to solve? I'm trying to write a solver for the weather nodes and it would be 1000x easier if I could grab a variable that is local to the Start() function.

Alexejhero commented 1 year ago

You cannot grab local variables using patches

ScrubN commented 1 year ago

You cannot grab local variables using patches

I could have phrased my question better. The collection needed is generated by a private method called by Start(). The collection has the solution to the maze, which is the final piece I need to finish that specific solver. I have checked the harmonypatch docs and I should be able to grab the collection.

I guess I was asking permission to incorporate a patch in my solver? If not then I would need to write a bunch of duplicate A code or make the existing A code more abstract to decouple it from the PathfindingHandler.

Alexejhero commented 1 year ago

Yeah you can definitely use patches if that's easier, seems like you can postfix SolveMaze and grab the hashset parameter from there

On Mon, Apr 17, 2023, 06:19 Scrub @.***> wrote:

You cannot grab local variables using patches

I could have phrased my question better. The collection needed is generated by a private method called by Start(). The collection has the solution to the maze, which is the final piece I need to finish that specific solver.

I guess I was asking permission to incorporate a patch in my solver? If not then I would need to write a bunch of duplicate A code or make the existing A code more abstract to decouple it from the PathfindingHandler.

— Reply to this email directly, view it on GitHub https://github.com/VedalAI/neuro-amongus/pull/37#issuecomment-1510636877, or unsubscribe https://github.com/notifications/unsubscribe-auth/AHV6XKCZSYCDEVRIB4JHWALXBSZE3ANCNFSM6AAAAAAWXLSM2A . You are receiving this because you were mentioned.Message ID: @.***>

ScrubN commented 1 year ago

No matter what I try I can't get the safe solver to be more consistent. I'm on the edge of just hard coding it to complete.