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

Fix transferring active Stim ability to Titan and endless stim effects #864

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

Conversation

Gazyi
Copy link

@Gazyi Gazyi commented Sep 4, 2024

In current version, if pilot has active Stim ability when embarking Titan, it'll be "transfered" to Titan. That's mostly noticeable when Stim has increased duration.
Tested with applied "bc_super_stim" mod. Notice Titan max velocity change after activating Stim:

NkH2YVR2.mp4

PR also applies fix for endless stim not having visual effects and sound not looping. However, there's still a problem with sound - sometimes even TF|1 looping sound stops working. I can't find a cause of this, but I suspect there's a limit of active sounds on entity, because it stops playing after pilot received a lot of damage.

@GeckoEidechse
Copy link
Member

This adds a new modified vanilla file, right? ^^

@GeckoEidechse GeckoEidechse added needs testing Changes from the PR still need to be tested needs code review Changes from PR still need to be reviewed in code labels Sep 4, 2024
@Gazyi
Copy link
Author

Gazyi commented Sep 4, 2024

This adds a new modified vanilla file, right? ^^

Yes.

@GeckoEidechse
Copy link
Member

Which VPK/path? Cause then I fixup the PR to add that file first and then layer change on top of it. That way it's easier to see what got changed and gets the PR merged faster :D

@Gazyi
Copy link
Author

Gazyi commented Sep 4, 2024

Which VPK/path? Cause then I fixup the PR to add that file first and then layer change on top of it. That way it's easier to see what got changed and gets the PR merged faster :D

englishclient_mp_common.bsp.pak000_dir.vpk/scripts/vscripts/weapons/sh_stim.gnut

@GeckoEidechse GeckoEidechse self-assigned this Sep 4, 2024
@Zanieon Zanieon added the commits vanilla file For PRs that commit vanilla files from VPKs label Sep 4, 2024
@GeckoEidechse GeckoEidechse changed the title Fix transferring active Stim ability to Titan and endless stim effects. Fix transferring active Stim ability to Titan and endless stim effects Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
commits vanilla file For PRs that commit vanilla files from VPKs needs code review Changes from PR still need to be reviewed in code needs testing Changes from the PR still need to be tested
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants