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

Custom voice pack improvements #2143

Draft
wants to merge 2 commits into
base: Dev
Choose a base branch
from

Conversation

rrealmuto
Copy link

@rrealmuto rrealmuto commented Nov 17, 2023

This PR improves and streamlines the process of creating custom voice packs for OOTR.

Currently, custom voice packs are fairly limited in terms of what you can do. They only support the vanilla sampling rate and must not exceed the original SFX length. The workflow is also a bit cumbersome and the audio files need to be manually converted to binary files to inject into the ROM using some 3rd party tools.

This PR removes all restrictions on custom voices by using the improved audio system from the MMR Music PR #2044 . Audio files must simply be mastered in .AIFF format and converted to .AIFC (the file format used by the N64) using the original N64 utility vadpcm_enc with the vanilla code book file. Will eventually write up a full procedure on creating custom voice packs in this format, but it is quite a bit easier than the current method.

Leaving this as draft until MMR music gets merged but just wanted to put it on people's radar.

Still to do:

  • Merge MMR Music PR
  • Clean up this PR
  • Document new music workflow
  • Build vadpcm_enc for windows (currently only works for linux) and maybe include the binaries in the OOTR repo?

@fenhl fenhl added Type: Enhancement New feature or request Component: Cosmetics Affects the patching of cosmetics labels Nov 17, 2023
@fenhl fenhl added Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested labels Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Component: Cosmetics Affects the patching of cosmetics Status: Needs Review Someone should be looking at it Status: Needs Testing Probably should be tested Type: Enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants