The Merchant's Tale (Advanced) - DaryaTheSeaMaid #744
Open
Equilius wants to merge 15 commits into
Open
Conversation
- Implement enums for the module - Implement first ability which is a raidwide
Implemented the following mechanics - FamilarCall - SunkenTreasures - Hydrobullet
- Implemented Hydrocannon - Implemented AquaSpear - Implemented SeaShackles - Implemented TidalWave
- Implemented SwimmingInTheAir - Implemented CeaselessCurrent - Implemented SurgingCurrent
- Implemented AlluringOrder - Implemented AquaBall - Implemented RecedingTwinTides
- Fixed AquaSpears tiles not despawning - Removed states from Enums.cs since I see we shouldn't store these since they might not be unique to the name given
- Fixed tethers not disappearing after they have been resolved - Fixed AquaSpear tiles not showing up until they actually casted
- Added a guard between pulls for the boss, if the party wipes while tethers are active they don't despawn in-game fully - Increased knockback distance for AI pathfinder
Author
|
The module has been completed; I tested it in a 4-man party with AI on, and it worked fine. Just wanna test it in a 2-man party since some mechanics get changed around, and if that works fine, then I will change it from draft to ready for review. |
- Increased the tether distance requirement between players to allow for movement updates without breaking the tether
Just an extra 0.5f to be safe, but it was working fine with 25.0f you were just a pixel away from being outside the bounds of the map
- If a player somehow dies before their spread icon goes off, they will keep the spread on them due to them missing the EventCast
- Implemented anti KB setting if the player is not able to get to their safe spot in time - Implemented corner safe spots for AlluringOrder which followed the style from TOPConfig
Author
|
Going to stop adding to this now; I just wanted to learn about using AI for a module, and I think I went a bit overkill with it since no one is going to use AI for this anyway, since it is a matchmade instance |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Hey, started working on DaryaTheSeaMaid (advanced) since I've seen people ask about it every so often. Once it is ready, I will mark it as ready for review.