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

Change: Add upgrade voices for Satellite Hack 1 and 2 #1262

Merged
merged 6 commits into from
May 7, 2023

Conversation

Stubbjax
Copy link
Collaborator

The Satellite Hack upgrades now each have their own EVA audio events, instead of the generic "Upgrade complete" line.

SH12.mp4

@Stubbjax Stubbjax added Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker labels Sep 23, 2022
@ImTimK
Copy link
Collaborator

ImTimK commented Sep 23, 2022

Sexy

@xezon
Copy link
Collaborator

xezon commented Sep 23, 2022

Are the sounds in their respective languages? Also what happens if USA player has Internet Center? Do we expect USA EVA voice as opposed to China EVA voice?

@Stubbjax
Copy link
Collaborator Author

Are the sounds in their respective languages? Also what happens if USA player has Internet Center? Do we expect USA EVA voice as opposed to China EVA voice?

Hmm, that's a good question. How does it work for the other EVA events that @commy2 added in #1026? The localised variants would have to be done by those familiar with those languages to ensure the correct sentence structure, assuming the respective audio exists for them.

@xezon
Copy link
Collaborator

xezon commented Sep 23, 2022

German, French, Russian can be sourced from locales.

Copy link
Collaborator

@commy2 commy2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

neat 👍

@xezon
Copy link
Collaborator

xezon commented Sep 23, 2022

Please do not merge until sounds are sorted.

commy2
commy2 previously requested changes Sep 23, 2022
Copy link
Collaborator

@commy2 commy2 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sort out sounds first

@Stubbjax
Copy link
Collaborator Author

Stubbjax commented Sep 23, 2022

Aha I wasn't planning to. Just review the PR as Changes Requested! (Looks like I was beaten to it.)

@xezon
Copy link
Collaborator

xezon commented Sep 23, 2022

Additionally, I think we should isolate this change first:
71f3bed#diff-e8bc6bb844a8f9f70ca66bb955fa3c2cafd736d83bc906c104bc15fdbc942cca

It simplifies how sounds are sourced to avoid English copies. We can extract this commit from #1061 and submit first. If you cannot do this on your own, let me know I will do it.

@Stubbjax
Copy link
Collaborator Author

Ah right, didn't see that. It did feel very dirty copying the files, but #1026 was approved and merged that way so I matched the process. Is #1061 awaiting anything in particular?

@xezon
Copy link
Collaborator

xezon commented Sep 23, 2022

#1061 has not been reviewed. To review, one needs to listen to 1000 sounds. I doubt anyone will do that, so eventually I will just merge it in the future. We can isolate that one commit from #1061, as they do not need to be in one Pull.

@Stubbjax
Copy link
Collaborator Author

Alright, so to clarify, cherry-pick commit 71f3bed#diff-e8bc6bb844a8f9f70ca66bb955fa3c2cafd736d83bc906c104bc15fdbc942cca and remove the duplicate audio files from the non-English localisation directories?

@xezon
Copy link
Collaborator

xezon commented Sep 23, 2022

Yea, that commit can be its own PR. Once that is committed, we do not need to make English copies.

@Stubbjax
Copy link
Collaborator Author

Yea, that commit can be its own PR.

You will open a dedicated PR?

@xezon
Copy link
Collaborator

xezon commented Sep 24, 2022

Commit #1273 before this one.

Also regarding English sounds, we should use the cleaned sounds from #1061, not the original ones.

@commy2
Copy link
Collaborator

commy2 commented Sep 24, 2022

lgtm

@xezon
Copy link
Collaborator

xezon commented Sep 24, 2022

It is still missing German, French, Russian, new English. I can add it if no one beats me to it.

@xezon
Copy link
Collaborator

xezon commented Apr 11, 2023

Also what happens if USA player has Internet Center? Do we expect USA EVA voice as opposed to China EVA voice?

I think it is fine if voice is different. When buying Rocket Buggy Ammo as a USA for example then we also hear the GLA Buggy voice.

The Satellite Hack upgrades now each have their own EVA audio events, instead of the generic "Upgrade complete" line.

Where are these sound files sourced from? I can not find these in original game files.

@commy2
Copy link
Collaborator

commy2 commented Apr 12, 2023

Same place MOAB and Fortified Structures sound if from. It's in a different folder for some reason.

@xezon
Copy link
Collaborator

xezon commented Apr 12, 2023

I was unable to locate it in all folders. Therefore I also do not know where it is for the other languages. Perhaps it is under a different name? If there is no same audio for German and French then we cannot use this.

@xezon xezon changed the title Add: Upgrade voices for Satellite Hack 1 and 2 Change: Add upgrade voices for Satellite Hack 1 and 2 Apr 12, 2023
@xezon
Copy link
Collaborator

xezon commented Apr 12, 2023

Here are all the EC files I have access to:

ecaactpc.wav
ecabuink.wav
ecabuipc.wav
ecabuiss.wav
ecallreq.wav
ecallyu.wav
ecanklau.wav
ecasslau.wav
ecbasof.wav
ecbasun.wav
ecbeaco.wav
ecblackd.wav
ecbuica.wav
ecbuildg.wav
ecbuilo.wav
ecbuiof.wav
ecbuion.wav
ecbuist.wav
eccapbla.wav
eccapbui.wav
eccapcol.wav
eccapher.wav
eccapjar.wav
eccashs.wav
eccolbud.wav
ecconst.wav
ecdeant.wav
ecdedai.wav
ecdeemp.wav
ecdefue.wav
ecdenuk.wav
ecdepar.wav
ecdescu.wav
eceblacd.wav
ececolbd.wav
ecejarmd.wav
ecempmup.wav
ecfunds.wav
eciobecl.wav
eciocacs.wav
eciocril.wav
eciodecs.wav
eciodesg.wav
eciodess.wav
eciodest.wav
eciofall.wav
ecioproc.wav
eciorise.wav
ecioverl.wav
ecjarmkd.wav
eclagpss.wav
eclanuk.wav
eclapar.wav
eclascu.wav
eclasnea.wav
eclevel.wav
ecneutup.wav
ecnuklau.wav
ecnukrea.wav
econline.wav
ecparact.wav
ecparrea.wav
ecpldef.wav
ecplvic.wav
ecpower.wav
ecrecfnd.wav
ecreinf.wav
ecscdlau.wav
ecscurea.wav
ecselec.wav
ecstrong.wav
ectraing.wav
ectrnfnd.wav
ecunitl.wav
ecunitp.wav
ecunitr.wav
ecupgra.wav
ecvehst.wav
ecyoureq.wav

@xezon xezon added the Audio Is audio related label May 7, 2023
@xezon xezon added the China Affects China faction label May 7, 2023
@xezon xezon force-pushed the satellite-hack-eva-audio branch from 204e9c4 to 4f66c07 Compare May 7, 2023 13:57
@xezon
Copy link
Collaborator

xezon commented May 7, 2023

I have remade the English sounds. And made French, German ones too. And added default Upgrade sound for Russian. Documentation added. Sounds good.

@xezon
Copy link
Collaborator

xezon commented May 7, 2023

Fixed one small error in French sound.

@xezon xezon merged commit ece2b4e into main May 7, 2023
@xezon xezon deleted the satellite-hack-eva-audio branch May 7, 2023 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Audio Is audio related China Affects China faction Enhancement Is new feature or request Minor Severity: Minor < Major < Critical < Blocker
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants