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

Sounds: fix persistent weather effects #59188

Merged
merged 1 commit into from
Jul 13, 2022

Conversation

dseguin
Copy link
Member

@dseguin dseguin commented Jul 12, 2022

Summary

None

Purpose of change

Weather sound effects (WEATHER_*) do not fade out like other environment variants like daytime, so the audio channel will continue to play until a different weather SFX is encountered.

Describe the solution

Fade out outdoor* channels when player is deaf or indoors, or if weather is silent.

Describe alternatives you've considered

I thought initially that there might be an issue with find_random_effect(), but it seems to be selecting the appropriate effects.

Testing

Following the steps in the linked issue:

  • Using CC-Sounds
  • Walk indoors from outside
  • WEATHER_CLEAR effect fades out

Tested in-game audio:

2022-07-11.21-30-53.mp4

Additional context

@github-actions github-actions bot added [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions labels Jul 12, 2022
@kevingranade kevingranade merged commit b3106cf into CleverRaven:master Jul 13, 2022
@dseguin dseguin deleted the weather_sfx_fix branch July 13, 2022 00:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
astyled astyled PR, label is assigned by github actions BasicBuildPassed This PR builds correctly, label assigned by github actions [C++] Changes (can be) made in C++. Previously named `Code` json-styled JSON lint passed, label assigned by github actions
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Outside sounds don't stop when you get inside
2 participants