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.

Sign up for GitHub

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

Jump to bottom

feat: Overhaul BaseAI Movement, AI, Bug Fixes, and Performance #2101

Closed
wants to merge 98 commits into from
Closed

feat: Overhaul BaseAI Movement, AI, Bug Fixes, and Performance #2101

wants to merge 98 commits into from

Conversation

Bohicatv
Copy link
Contributor

@Bohicatv Bohicatv commented Feb 1, 2025

No description provided.

Bohicatv and others added 30 commits January 18, 2025 21:33
@Bohicatv
Update BaseAI.cs - full refactor/overhaul
e38cc52
@Bohicatv
Update BaseAI.cs
a7aa07a
@Bohicatv
Update BaseAI.cs
296fa9c
@Bohicatv
Update BaseAI.cs - Remove custom enum types
4a66e17
@Bohicatv
Update BaseAI.cs - Restore mobile speed assignments
fafd352
@Bohicatv
Update BaseAI.cs - Add combat delay
cbd0a2a
@Bohicatv
Update BaseAI.cs -- ShouldApplyCombatDelay method
8253bc4
@Bohicatv
Update BaseAI.cs - Hides invalid mobiles instead of deletion
a3f0722
@Bohicatv
Update BaseAI.cs - Adds "invalid" prefix if mobile becomes invalid
43bb57a
@Bohicatv
Consolidate IsValidTarget method
5305c44
@Bohicatv
Consolidate CanAttackTarget Method
d13a3a8
@Bohicatv
Add Cliloc in HandleMove method
1c22536
@Bohicatv
bugfix/Fixes mobs from becoming frozen after combat
1878bfa
@Bohicatv
Refactor IsValidCombatant method
a1c6b0d
@Bohicatv
Refactor/Deprecate DoActionGuard/FindGuardTarget method
5811b8b
@Bohicatv
Refactor DoOrderNone
67aae98
@Bohicatv
Deprecate IsValidCombatant method
92af0bb
@Bohicatv
Refactor HandleLostMaster method
28893df
@Bohicatv
Rafactor HandleComeOrder method
ee0bd17
@Bohicatv
Add RefreshCombatantState method
77f856b
@Bohicatv
Refactor DoOrderDrop method
2b8e09f
@Bohicatv
Remove comments from CheckHerding method
9aa3cb3
@Bohicatv
Refactor DoOrderFollow method
1460241
@Bohicatv
Refactor IsInvalidFriendRequest method
4de2253
@Bohicatv
Refactor IsInvalidUnfriendRequest method
77fe34c
@Bohicatv
Add FindCombatant method
a24647b
@Bohicatv
Refactor FindNewCombatant, Deprecate other methods
1206e42
@Bohicatv
Refactor DoOrderTransfer
2ba3247
@Bohicatv
Refactor IsInCombatState
2ec72bc
@Bohicatv
Refactor HandleLostProvoker
c7f047a
Bohicatv and others added 25 commits January 27, 2025 08:54
@Bohicatv
Merge branch 'main' into Refactor/BaseAI-Movement-Bugfixes
6c4c522
@Bohicatv
fix: Prevents unnecessary speed updates in AI order handling
f11deb1
@Bohicatv
fix: Prevents redundant action assignments in AI behavior
e31c4f2
@Bohicatv
fix: Enhances null checks and movement conditions in AI logic
084cce3
@Bohicatv
refactor: Remove redundant debug logging in DetectHidden method
12d4160
@Bohicatv
refactor: Simplify bad state checks
226d7a0
@Bohicatv
refactor: Optimize target validation checks in AcquireNewFocusMob method
f15e2eb
@Bohicatv
refactor: Improve interval update logic and streamline OnTick method
b3628df
@Bohicatv
refactor: Remove unused CheckLeashed method and clean up OnTick logic
f87f5f0
@Bohicatv
refactor: Simplify action handling logic in BaseAI class
7a664ea
@Bohicatv
refactor: Enhance order handling by adding conditional checks for sta…
000f68b 
…te changes
@Bohicatv
refactor: Add conditional checks to reset mobile control properties i…
da84888 
…n DoOrderFollow method
@Bohicatv
refactor: Add conditional checks to reset mobile properties in order …
2712c9b 
…handling methods
@Bohicatv
Merge branch 'main' into Refactor/BaseAI-Movement-Bugfixes
7ab865d
@Bohicatv
fix: Remove unused DebugSay method from BaseAI class
4fad46d
@Bohicatv
fix: Adjust GuardNothing method to set CurrentSpeed only if not alrea…
c913207 
…dy at 0.1
@Bohicatv
Comment Cleanup
9892047
@Bohicatv
revert: Debugsay method (removed by mistake)
67d94cc
@Bohicatv
fix: Include maxReduction parameter and streamline speed calculations
73294c8
@Bohicatv
refactor: Simplify CheckFlee
6fdb609
@Bohicatv
balance: Reduce mobile movements when idle/wander
5260c7f
@Bohicatv
chore: Remove development disclaimer
80bb749
@Bohicatv
balance: Adjust the rest of the random movement parameters for AI
200b98d
@Bohicatv
fix: Validate snapshotPath in WriteFiles method to prevent null argum…
eea2de7 
…ents
@Bohicatv
fix: Validate savePath in WriteSnapshot method to prevent null arguments
7b7706e
@Bohicatv Bohicatv changed the title fix: Ensures world save paths are validated feat: Overhaul BaseAI Movement, AI, Bug Fixes, and Performance Feb 1, 2025
@Bohicatv Bohicatv closed this Feb 1, 2025
@Bohicatv Bohicatv deleted the bugfix/WorldSave-Path-Validation branch February 1, 2025 22:42
@kamronbatman
Copy link
Contributor

Why was this closed?

@Bohicatv
Copy link
Contributor Author

Bohicatv commented Feb 2, 2025

eea2de7
7b7706e
Commits were supposed to be in a separate PR, but my IDE included 2 branches by mistake.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Bohicatv @kamronbatman