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

Variable NPC Juke Aggression #935

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

VMSolidus
Copy link
Member

@VMSolidus VMSolidus commented Sep 20, 2024

Description

This PR un-hardcodes the JukeSystem timer, such that individual NPC HTN Blackboards can directly state how often, and how far they wish to juke. The effect of this is that NPCs are no longer completely impossible to hit with left-clicks in melee combat, while also allowing for more "Elite" enemies that juke more aggressively and more often to be created.

Additionally, by introducing an exit condition based on this new "Juke Cooldown", I have made the Juke Operator run a metric shitload of expensive calculations 5000 times less often.

Media

Melee enemy with the new default juke settings, 0.5 second juke duration, with 5 second juke Cooldown. The reagent slime will attempt to close to melee with its enemy, but will now only attempt to evade melee attacks once per 5 seconds. They will otherwise attempt to stay within melee range.

8mb.video-gfc-C8xLWRyh.mp4

Changelog

🆑

  • tweak: JukeOperator now allows for JukeDuration and JukeCooldown arguments. JukeCooldown is a new feature where enemies must wait an amount of time in seconds equal to the JukeCooldown, before they are allowed to attempt to dodge a melee attack.
  • tweak: By default, NPCs will only attempt to dodge attacks once every 5 seconds.
  • fix: JukeOperator now performs extremely expensive math 5000 times less often. EXIT CONDITIONS PEOPLE!

@github-actions github-actions bot added the Changes: C# Changes any cs files label Sep 20, 2024
@FoxxoTrystan FoxxoTrystan requested review from a team, DEATHB4DEFEAT, Pspritechologist and FoxxoTrystan and removed request for a team September 20, 2024 01:33
@github-actions github-actions bot added the Status: Needs Review Someone please review this label Sep 20, 2024
@FoxxoTrystan FoxxoTrystan added Priority: 4-Low Should be resolved at some point Size: 4-Small For small issues/PRs labels Sep 20, 2024
@VMSolidus VMSolidus merged commit 104ee3e into Simple-Station:master Sep 20, 2024
17 checks passed
SimpleStation14 added a commit that referenced this pull request Sep 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Changes: C# Changes any cs files Priority: 4-Low Should be resolved at some point Size: 4-Small For small issues/PRs Status: Needs Review Someone please review this
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants