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

Mahoyo voice fixes #6

Open
wants to merge 22 commits into
base: master
Choose a base branch
from

Conversation

FallenStar08
Copy link

@FallenStar08 FallenStar08 commented Jul 30, 2023

Bonjour, voilà ma PR qui constate quelques (beaucoup) de problèmes avec le patch des voix éliminés !

  • Le plus gros changement concerne le fait qu'au lieu du marqueur générique @sestop avant chaque ligne de dialogue, j'ai créé un petit script qui arrête uniquement la ligne audio de dialogue spécifique qui jouait avant à chaque fois. De ce fait, il n'y a plus de coupure des autres sons !

Viennent ensuite les changements plus spécifiques, dans l'ordre et, si je n'ai rien oublié normalement (les changements en gras doivent probablement aussi être effectués sur le patch de base, car ils ne sont pas liés au patch voix) :

  1. Dans 1.5-4.ks, à partir de la ligne 642, les lignes audio sont dans le désordre (décalées du moins). J'ai manuellement réglé le problème (cela provient du premier "Bonjour" qui utilise le même audio que la ligne suivante ; les lignes ont dû être fusionnées dans la version switch).
  2. Dans 2-4.ks ligne 356 ajout d'un [l] pour ne pas se faire couper par la ligne audio suivante
  3. Dans 4-1.ks ligne 228 Aoko a une ligne de dialogue en trop est qui est du coup mal placée, cette ligne semble dire quelque chose du genre "ça se passe comme prévu" et devrait donc avoir le texte correspondant avant "Demain, on aura considérablement réduit le nombre de suspects" la ligne audio en question est A10_4_1_0002.opus, j'ai retiré ce son donc et je vous laisse décider d'ajouter la ligne en texte ou non.
  4. Dans 4-2.ks on a un "*page14" dans le texte, aussi présent dans le patch de base, Cependant, j'ai seulement touché au patch8, donc je vous laisse gérer cela.
  5. Dans 5a-8.ks on a A10_5A_8_0018.opus qui joue au mauvais endroit. Je l'ai déplacée là où il devrait être (avant la ligne 1251).
  6. Dans 6-6.ks ligne 553 simple ajout d'un [l] pour ne pas se faire couper par la ligne audio suivante
  7. Dans 7-2.ks B60_7_2_0000.opus déplacé avant la ligne 80 à la place
  8. Dans 8-6.ks ligne 335 simple ajout d'un [l] pour ne pas se faire couper par la ligne audio suivante
  9. Dans 8.5.ks ligne 2936 - suppression du [l] en début de ligne, ajout d'un [l] ligne 3232 et correction d'une faute de frappe à la ligne 3410 (cel->cela).
  10. Dans 9-3c.ks ligne 626 ruby text en japonais supprimé
  11. Dans wik_wakeup.ks suppression des [l] ligne 128 & 129 pour que l'audio de Ritsuka joue correctement avec le texte et son animation
  12. wik-g-a.ks ligne 332 simple ajout d'un [l] pour ne pas se faire couper par la ligne audio suivante
  13. nz6.ks à partir de la ligne 320 environs, grosse remise en ordre des lignes audios causée par un décalage semblable à celui de 1.5-4.ks

J'ai probablement omis quelques petits changements, comme des lignes audio qui jouaient au milieu du texte et non au début, mais cela devrait être visible dans les commits.

Voilà, j'ai joué entièrement avec la grosse modification et j'ai également vérifié les remises d'ordre des lignes audio après coup ; aucun problème particulier n'a été trouvé !

Désolé pour les commits nommées à la va-vite...

Il est fort probable qu'un petit problème ou deux subsistent, mais je pense avoir capturé l'essentiel !

@IDerr
Copy link
Owner

IDerr commented Jul 31, 2023

Superbe! J'essaie de review ça auj ou demain!

@IDerr
Copy link
Owner

IDerr commented Aug 8, 2023

J'ai ete trop optimiste, ca sera ce weekend...

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