Conversation
Could also be possible to do a pattern output or ice => fire => lightning like output, but length of TTS could be noisy
For the players that have a debuff, we can call the general area ~2s prior
Wrapper outputString provided that would let player alter string based on debuff.
|
I'm not sure how helpful the Exa trigger location can be. This mechanic may be too fast for the output to be meaningful and there may not be a predictable pattern to merge every other one. Also the output could remove NW/NE and instead use something like a number count as odds are always nw and evens are always ne. I guess ideally to make any actual use of it, the player would need to override the pattern matching with the markers they are using. |
was seeing emulator skip first one and timings be off in emulator
|
Example outputs (this is with the P4/P5 ultima upsurge split) from the other PR #1117: |
Co-authored-by: valarnin <valarnin@gmail.com>
|
ah, that comment is actually just wrong for Clockwise Example 2, I see it's just a copy of the x,y from previous line so will need to go back and check it. EDIT 2: I see there is also another typo in Counterclockwise Example. |
|
Is there an update to when this could be merged, if the issues are resolved? |
There was a problem hiding this comment.
I reviewed the requested changes.
The ternary operator in Fell Forces trigger was changed to if statements.
The triggers for Flood needed more than heading data and I found the x, y data to not be correct in the StartsUsingExtra. However, in my review of it I realized the second output could be updated to be more accurate than just the intercardinal start and so it now outputs the cardinal position which will be closer to the second flood and direction to move. This change matches the kefkabin guide for starting position.
The comments on Suprise Flare/Holy have been corrected.



This PR adds triggers for P5 mechanics: