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

[PVP] Global Emergency Guard Anti Mash edit #154

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

edewen
Copy link
Contributor

@edewen edewen commented Dec 1, 2024

Edited the Anti mash feature.

Original feature prevents you from unguarding by accident in pvp using your combo buttons. It only leaves guard untouched so you can use that to get out of guard.

The change

If you have the mp AND could use the 15k healing with wasting it, it changes that guard button to recuperate so that can be used to take you out of guard. Exact same functionality, removing guard, but allows you to do it with the heal instead of having to hit guard and then heal resulting in a delay.

The original request in discord wanted other skills, but to do that, would require unlocking those skills under guard and would likely result in the anti-mash feature having potential fails.

@Kaeris-Tempest
Copy link
Contributor

This will cause Guard to break immediately if you're mashing and missing health.

@zbee
Copy link
Member

zbee commented Dec 1, 2024

If you're mashing guard, yes it would, but not if you entered Guard via the Emergency Guard Feature (i.e. pressing your main combo).

Not sure what else it could or should go on.

I think the answer would be to make this its own setting, with a slider of how much time left on the Guard buff is required before triggering the feature (e.g. make the buff go on for at least 2s before letting you exit via Recuperate - e.g. GetBuffRemainingTime(Buffs.Guard) < 4 would make it require Guard be on for a full second before replacing with recup)

@zbee zbee added Optimization Similar to suggestion, but solely for increasing performance New Feature This adds a new option or feature labels Dec 1, 2024
Added just used 2f to keep it from letting you break a bit too early if you are somehow spamming the guard button not the combo button.
@edewen
Copy link
Contributor Author

edewen commented Dec 1, 2024

get buff is too slow, i still see a flash of recup before it goes back to guard which COULD be bad for fast button spammers, though unlikely. Went with !justused guard and a 2f to keep it from showing up too quickly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
New Feature This adds a new option or feature Optimization Similar to suggestion, but solely for increasing performance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants