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

Add warnings to the Teleport Hack option #455

Merged
merged 1 commit into from
Nov 21, 2024

Conversation

zbee
Copy link

@zbee zbee commented Nov 21, 2024

Add warnings and more verbosity to try to deter public use of this feature.

Per discussion on #451 I specifically tried to align with how other contributors explained this option should be used.

@CarnifexOptimus
Copy link

you do seem really passionate about this topic

@zbee
Copy link
Author

zbee commented Nov 21, 2024

I am very much opposed to it yes; I assumed a removal PR had no chance, and that this may be merged and mitigate some of my worries.

@CarnifexOptimus CarnifexOptimus merged commit 34313e3 into FFXIV-CombatReborn:main Nov 21, 2024
@CarnifexOptimus
Copy link

Since you are a software engineer you could help to make a proper solution for temporary misdirection and spinning (b3 alzaadals legacy) so these kind of hacks are not needed

@zbee
Copy link
Author

zbee commented Nov 22, 2024

Thanks for the merge.
I appreciate the suggestion; if the need is enough for this then yeah, I'll try to carve some time to contribute upstream to VBM 👍🏻

@CarnifexOptimus
Copy link

Would definitely be appreciated if something less cheaty than noclip or teleport hack could be used for this. But i believe the client structs have not reverse engineered these debuffs yet

@zbee
Copy link
Author

zbee commented Nov 23, 2024

The author of that mod was already on it when I asked, so I waited. There's a run-down of it below, hopefully that can replace the option I was adding a warning to here.

Check for misdirection here.

Then read the misdirection like so:

public static readonly float* ForcedMovementDirection = (float*)Service.SigScanner.GetStaticAddressFromSig("F3 0F 11 0D ?? ?? ?? ?? 48 85 DB");

And finally account for it here:

var currentDir = Angle.FromDirection(new(*sumLeft, *sumForward)) + ForwardMovementDirection();
var dirDelta = currentDir - ForcedMovementDirection->Radians();

@CarnifexOptimus
Copy link

CarnifexOptimus commented Nov 23, 2024

can you ask if it also works for the spinning debuff in alzaadals legacy? it works a little different and i am too lazy to check right now (status IDs 2973 and 3702)

@zbee
Copy link
Author

zbee commented Nov 23, 2024

I would assume not, as it explicitly checks for if (sm->Status[i].StatusId is 1422 or 2936 or 3694 or 3909).
Though, unless I misremember, I think that one is very different; I'll test and update this comment.

@CarnifexOptimus
Copy link

Probably not then, but maybe you can bring those status effects to his attention. Veyn blocked me some time ago so I can't do it.

@zbee
Copy link
Author

zbee commented Nov 23, 2024

Yes, it does, which is interesting.
image

@CarnifexOptimus
Copy link

I wonder if it just detects the misdirection or if it can actively handle it? Afaik for temporary misdirection you have to wait until the hand points into the right direction while during spinning you can influence the direction somewhat

@zbee
Copy link
Author

zbee commented Nov 23, 2024

Would have to recommend looking at what I linked when you have the time.
The third link I provided is what you could use for that. In VBM it currently just prevents you from providing input when that angle+your camera (accounting for movement settings) would result in you moving askew, for you, you could read once that's +-5 degrees or something, then move the player, for example.

I'll leave it there for now, since the conversation maybe isn't suited to this PR.
But if you would like my help to develop a feature to replace the teleport hack options then you could ping me in an Issue-task once you have had a chance to look it over and have decided how you would like it implemented.

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

Successfully merging this pull request may close these issues.

2 participants