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

Multishot weapons #419

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

Multishot weapons #419

wants to merge 3 commits into from

Conversation

bboudaoud-nv
Copy link
Collaborator

This branch adds a pelletsPerShot parameter to weapons, where all pellets are fired at the same simulation step and each pellet does damagePerSecond * firePeriod damage to each target it hits. This is to emulate "shotgun-style" weapons which still have a large cooldown but fire multiple pellets in every shot. This change should work for both hit scanned and propagated projectiles.

Due to the nature of the changes in this branch (iterating in the Weapon::fire() routine) we should be careful it doesn't have performance impacts for large target/projectile counts and document where it does. The current implementation does run a nested loop that is # of pellets * # of targets total iterations.

@bboudaoud-nv bboudaoud-nv added the enhancement New feature or request label Jan 19, 2023
@bboudaoud-nv bboudaoud-nv requested a review from jspjutNV January 19, 2023 22:14
@bboudaoud-nv bboudaoud-nv self-assigned this Jan 19, 2023
@bboudaoud-nv
Copy link
Collaborator Author

Before merging this branch we should figure out how we want to log the results of multi-pellet shots

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant