-
Notifications
You must be signed in to change notification settings - Fork 48
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
Minigame solvers #37
Minigame solvers #37
Conversation
@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. |
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
This comment was marked as outdated.
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
<#37 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AHV6XKCZSYCDEVRIB4JHWALXBSZE3ANCNFSM6AAAAAAWXLSM2A>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Requesting changes for 3 solvers, if you do not want to change some of them feel free to remove them from the PR and someone else will take them.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This doesn't look that good, the motion looks very unnatural and at the end it just snaps to the destination even if it never saw it.
Maybe we need to implement some kind of curved movement for the cursor for tasks like this one and upload data on airship
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I adjusted the movement a bit and it should be a bit smoother.
I do agree, a slerp based movement or similar could definitely look a little nicer.
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. |
I got bored of doing locomotion so I wrote some solvers.
Currently implemented: