Skip to content
This repository was archived by the owner on Dec 4, 2020. It is now read-only.

Ranged Attacks for Trusts & Trust: Semih Lafihna & Ayame WS Logic #1424

Merged
merged 13 commits into from
Nov 3, 2020
Merged

Ranged Attacks for Trusts & Trust: Semih Lafihna & Ayame WS Logic #1424

merged 13 commits into from
Nov 3, 2020

Conversation

zach2good
Copy link
Contributor

@zach2good zach2good commented Oct 24, 2020

I affirm:

  • that I agree to Project Topaz's Limited Contributor License Agreement, as written on this date
  • that I've tested my code since the last commit in the PR, and will test after any later commits

TODO

  • Ranged WS animations for Semih are messed up.
        [3487] = {['name']="Sidewinder",                     ['category']=11, ['id']=3487, ['animation']=429,  ['message']=185, },
        [3489] = {['name']="Stellar Arrow",                  ['category']=11, ['id']=3489, ['animation']=430,  ['message']=185, },
        [3490] = {['name']="Lux Arrow",                      ['category']=11, ['id']=3490, ['animation']=431,  ['message']=185, },
  • Actionview shows the category, id, animation and message are all same between retail and my changes...
  • Model is correct, capped from retail...

DONE

  • Basic RA for trusts
  • AI for Semih
  • Trusts stop running are 3 ticks of being chased 💀
  • I think I'm going to have to move all the RA stuff down into BattleEntity, it just makes sense. Even if it's stubbed.
  • Sane damage and delay for RA
  • Throwing support for Joachim
  • Bindings to tell Trusts to use RA
    mob:addSimpleGambit(ai.t.TARGET, ai.c.ALWAYS, 0,
                        ai.r.RA, 0, 0) -- Try RA all the time, but bound by your weapon's delay

@ibm2431
Copy link
Contributor

ibm2431 commented Oct 24, 2020

I think I'm going to have to move all the RA stuff down into BattleEntity, it just makes sense.

I'm surprised it already isn't. Do Goblin Ambushers currently not Ranged Attack?

@TeoTwawki
Copy link
Contributor

There are 2 types of ranged attack mob type objects can use. one is a "real" ranged attack, the other is a mobskill faking a ranged attack. A case of SE having multiple devs who didn't communicate to each other how they implemented. In game on retail, you can tell them apart by how the message displays: 1 line of text with no line break, or 2.

both types should be handled at battelentity imo.

@zach2good zach2good marked this pull request as ready for review November 1, 2020 14:53
@zach2good zach2good changed the title [WIP] Ranged Attacks for Trusts & Trust: Semih Lafihna Ranged Attacks for Trusts & Trust: Semih Lafihna Nov 1, 2020
@zach2good zach2good changed the title Ranged Attacks for Trusts & Trust: Semih Lafihna Ranged Attacks for Trusts & Trust: Semih Lafihna & Ayame WS Logic Nov 2, 2020
@zach2good zach2good merged commit 76a3b3a into project-topaz:feature/trust Nov 3, 2020
@zach2good zach2good deleted the trust_ranged branch November 3, 2020 05:51
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants