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

weight component #291

Closed
ozkriff opened this issue Jun 26, 2018 · 3 comments · Fixed by #621
Closed

weight component #291

ozkriff opened this issue Jun 26, 2018 · 3 comments · Fixed by #621

Comments

@ozkriff
Copy link
Owner

ozkriff commented Jun 26, 2018

#286 (comment)

shmutalov:

Will bomb explosion push neighbor bombs?

ozkriff:

yep, it's really strange that pushbacks don't interact with other bombs. Atm, they only push objects with agent component.

Though I don't want to push boulders around so easily, so I need to introduce something like weight component and only push "light" objects. I'll create another issue for that.


Weight table:

  1. normal objects (examples: normal agents, bombs)
  2. heavy objects (examples: heavy agents, boulders)
  3. totally immovable objects

Obviously, push effect should get similar "strength" parameter.

@ozkriff
Copy link
Owner Author

ozkriff commented Jun 26, 2018

Thought: maybe just add a field to blocker component?

@Vrixyz
Copy link
Contributor

Vrixyz commented Nov 14, 2019

I might give this issue a try, in the meantime I'm writing here what I understand from the possible impacts, for future reference or if someone else wants to tackle that:

  • push effect from this issue description actually refers to "Knockback" effect in the code
  • Strength parameter would be implemented the same way as "Heal" ability
  • Weight component (or blocker property) would be/contains an enum to match the specified weigth table

@ozkriff
Copy link
Owner Author

ozkriff commented Nov 15, 2019

... I'm writing here what I understand from the possible impacts ...

I haven't fleshed the exact plan for this feature out yet, but this looks correct.

I might give this issue a try ...

Great! Feel free to DM me on Discord or gitter if you want to discuss any details!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants