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

Added Requested Feature: Writer Audio lerp toggle #934

Closed
wants to merge 3 commits into from

Conversation

Arylos07
Copy link
Contributor

Description

Added a toggle to Writer Audio so the Writer does not lerp audio while writing. This results in the volume being consistent while writing.

Default and prefab Say Dialogs have this enabled so they will lerp audio by default. The user must change the prefab settings or create a custom Say Dialog to make use of this change.

What is the current behavior?

Issue Number: #929

When using a sound such as typewriters or other effects, the lerping audio can cause the sound to change volume rapidly and not sound consistent. This can be replicated by doing the following:

  • Open the WritingSpeedTest scene
  • Adjust the writers on the SayDialogs
    • Change beep sounds to contain only the MidVoice sound effect.
  • Play the scene.

The sound modulates heavily as it stops and stops. With the other voices, it's not heard, but with one consistent sound effect, the effect is noticeable and causes peaks in the audio levels.

What is the new behavior?

  • When Lerp Writer Volume is enabled, the default lerp volume behaviour remains
  • When Lerp Writer Volume is disabled, the writer remains at the target volume set by the Writer without lerping.

Important Notes

  • My change require modifcations or additions to documentation

    • Possibly, the variable may need to be added as well as a description of what it does. There is a tooltip in the editor that is thorough.
  • My change modifies the runtime execution/behaviour of existing Fungus Features. e.g., Say, Menus, Portraits, etc.

Other information

Setting in editor

This issue seemed simple to tackle and the feature would be nice to add depending on what audio you use. The dopplering effect is minor with the built-in sound clips, but amplified with things like typewriters.

* Added a Lerp Writer Audio bool.
  * When enabled (default), the WriterAudio uses the default behaviour of using MoveTowards for volume.
  * When disabled, the WriterAudio sets the audio volume to the target volume while writing, removing the lerp effect.

This setting is enabled by default on the say dialogue prefab Fungus spawns by default and therefore cannot be set at runtime.
@Arylos07 Arylos07 changed the title addressing issue #929 Added Requested Feature: Writer Audio lerp toggle #929 Dec 28, 2020
@Arylos07 Arylos07 changed the title Added Requested Feature: Writer Audio lerp toggle #929 Added Requested Feature: Writer Audio lerp toggle Dec 28, 2020
@stevehalliwell stevehalliwell added this to the 3.15 milestone May 10, 2021
@Arylos07
Copy link
Contributor Author

Closing this for now. Made a fresh develop branch and opening a new PR using that fresh and clean branch.

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

Successfully merging this pull request may close these issues.

2 participants