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

Paladin Taunt Skill - Avengers Shield #134

Open
wants to merge 23 commits into
base: main
Choose a base branch
from

Conversation

thmsndk
Copy link
Contributor

@thmsndk thmsndk commented Apr 22, 2024

This PR adds a taunt ability to the paladin

  • Requires shield
  • chains / bounces to two additional nearby targets from the initial target
  • taunts targets on hit
  • Does 10% attack damage too all targets hit

Additionally the equipped shield will be shown when throwing. So i'm adding projectile animations for each shield, as that was the easiest way to animate the shield in offhand without rewriting projectile rendering

Engine Changes

  • skills.py
    • can now define a attack_multiplier as well as a mp_cost_multiplier
    • can define if the skill is chained, it is only implemented in avengers shield skill
  • server.js
    • projectile_loop does not delete a projectile unless complete_attack returns true
    • commence_attack sets attack via gSkill.attack_multiplier
    • commence_attack sets mp_cost via gSkill.mp_cost_multiplier
    • commence_attack can now be instructed about chained as well as redirect that will be passed on to complete_attack
  • js/game.js
    • only delete a projectile from the client when it is no longer chained
  • js/html.js
    • show class specific skills even though you don't have the required weapon equipped

Vote for the skill

https://discord.com/channels/238332476743745536/238332476743745536/1231408673516032080
image

Shield Animations

  • wshield
    paladin-avengers-shield-wshield

  • shield
    paladin-avengers-shield-shield

  • mshield
    paladin-avengers-shield-mshield

  • tigershield
    paladin-avengers-shield-tigershield

  • xshield
    paladin-avengers-shield-xshield

  • sshield
    paladin-avengers-shield-sshield

if no entry is found the crusader shield is used as a default animation.
paladin-avengers-shield-crusader

@thmsndk thmsndk changed the title paladin avengers shield WIP paladin avengers shield Apr 22, 2024
@thmsndk thmsndk marked this pull request as ready for review April 23, 2024 19:27
@thmsndk thmsndk changed the title WIP paladin avengers shield Paladin Taunt Skill - Avengers Shield Apr 23, 2024
@melg8
Copy link

melg8 commented Apr 23, 2024

@thmsndk dont know if this was intentional changes?
node/server.js
100644 → 100755
js/game.js
100644 → 100755

Questions

  1. What happens if you use it while there is only 1 target? only 2 targets?
  2. What happens if there 3 targets and 1 hostile player nearby?
  3. What happens if there 2 targets and 1 hostile player nearby?
  4. Can it jump across different mob types (for example starting 2 jumps on goo, and hitting bee? - if its nearby?) or it only jumps over targets of same name?
  5. Does it correctly show skill on paladins skill menu and on wiki page of all skills?
  6. Can it be thrown while attack is on cooldown?
  7. Does it apply burn/freeze effects on jumped targets?
  8. Can you record something like 60 fps video of this spell maybe upload it onto youtube. Cause now i dont know if its skill that looks so shattery or is it just gifs with low fps showing it badly.
  9. Any ideas for custom sound for it?
  10. Any ideas how this skill can be abused by players with bad intentions? Griefers? Tripple pala setup for example?
  11. Plans on fixing this todo:? // TODO: if complete attack executes a "hit" (avoid,miss) earlier this part never executes, the projectile should probably be removed

Comments

  1. Crusader and gold shield gamma seems too bright as for me. Looks out of place.
  2. I would reduce size of its sprite like for 10-15% maybe.

@thmsndk
Copy link
Contributor Author

thmsndk commented Apr 24, 2024

@thmsndk dont know if this was intentional changes? node/server.js 100644 → 100755 js/game.js 100644 → 100755

Not sure what you think was unintentional?

Questions

  1. What happens if you use it while there is only 1 target? only 2 targets?

It only bounces to additional targets, if there are any

  1. What happens if there 3 targets and 1 hostile player nearby?
  2. What happens if there 2 targets and 1 hostile player nearby?

It only bounces between monsters currently

  1. Can it jump across different mob types (for example starting 2 jumps on goo, and hitting bee? - if its nearby?) or it only jumps over targets of same name?

Yes it can jump to different mob types

  1. Does it correctly show skill on paladins skill menu and on wiki page of all skills?

It should

  1. Can it be thrown while attack is on cooldown?

Yes, the skill has it's own cooldown as it is now

  1. Does it apply burn/freeze effects on jumped targets?

It should behave like other skills does, haven't tested

  1. Can you record something like 60 fps video of this spell maybe upload it onto youtube. Cause now i dont know if its skill
    that looks so shattery or is it just gifs with low fps showing it badly.

Not sure what you mean by shattery, it's pixel graphics after all :P

  1. Any ideas for custom sound for it?

Nope

  1. Any ideas how this skill can be abused by players with bad intentions? Griefers? Tripple pala setup for example?

Nope, i've not spent time thinking about abuse.

  1. Plans on fixing this todo:? // TODO: if complete attack executes a "hit" (avoid,miss) earlier this part never executes, the projectile should probably be removed

Think I did fix it by deleting the projectile when complete_attack returns true, would have to verify again

Comments

  1. Crusader and gold shield gamma seems too bright as for me. Looks out of place.

Probably because i've recorded it with HDR, HDR messes up gamma / brightness and I can't do anything about it, day/night cycles will also effect how it looks

  1. I would reduce size of its sprite like for 10-15% maybe.

@melg8
Copy link

melg8 commented Apr 24, 2024

@thmsndk maybe changes of filemodes was unintentional?

What will happen if pala is wearing exoskeletal arm?
Thanks for answers.
By shattery i mean i see like 2-3 frames of shield flying in one direction, then its gone, i don't see smooth trajectory of it moving towards first target, then second, then third.

@thmsndk
Copy link
Contributor Author

thmsndk commented Apr 24, 2024

What will happen if pala is wearing exoskeletal arm? Thanks for answers.

No idea, Not something i've tested, and i've never used the exosketal arm. What are your concerns?

By shattery i mean i see like 2-3 frames of shield flying in one direction, then its gone, i don't see smooth trajectory of it moving towards first target, then second, then third.

The projectile animation is not always correct, bouncing from 1 target to the next, I presume it is because the visuals are a little behind from the server, so the events does not happen exactly as expected. e.g. the server hit on mob #2 triggers before the projectile has reached mob #2 on the client and continues onwards to mob #3.
I'm not sure how easy thoose visuals are to solve without a lot of time investment. So for now i'm fine with how the animation looks. the tigershield animation is how it's supposed to look all the time

@melg8
Copy link

melg8 commented Apr 24, 2024

@thmsndk i mainly asked about this things for you and community to consider, so it doesn't just slip undercover. About exoskeletal - idk, just that it works properly, cause 2 out of 3 palas in characters screen seems to be using it. In general im okay with that change, although maybe it need some polish, thanks for your work for community.

@thmsndk thmsndk force-pushed the feature/paladin-avengers-shield branch from 7f17d5f to 9653380 Compare July 27, 2024 20:34
@Telokis Telokis added Type: feature Adds new content to the game Scope: game server The game server is impacted Status: waiting for review This PR is waiting for a maintainer to review it labels Aug 13, 2024
@thmsndk thmsndk force-pushed the feature/paladin-avengers-shield branch from 9653380 to f06fbac Compare August 13, 2024 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Scope: game server The game server is impacted Status: waiting for review This PR is waiting for a maintainer to review it Type: feature Adds new content to the game
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants