07th-mod / umineko-question

93 stars 9 forks source link

Voiced and Unvoiced Delay Bugs #92

Open drojf opened 6 years ago

drojf commented 6 years ago

the scene immediately following this line has the !s0 problem (text doesn't appear until line is finished)

She took it, ......and after hesitating, opened it...and read what was inside

langjp^^!s0 そして、真に謝るのは自分の方であることを自覚し、/
langjp^^!d800…/
langjp^^!d500…/
langjp^^!d500…/
langjp^^!d500…/
langjp^^!d500謝罪した。/
langen^^!s0^Then, realizing that she was the one who should be apologizing, ^/
langen:delay 1600^...^/
langen:delay 1000^...^/
langen:delay 1000^...^/
langen:delay 1000^...^/
langen:delay 1000^she apologized.^/

On chiru, the bug only affected voice lines? but maybe that was just a coincidence and there was something else breaking the automatically displaying text lines.

Also, the delays haven't been adjusted? They seem too short most of the time.

Edit: somehow changing the 'delay [delaytime]' commands to '!d[delaytime]` makes it show up correctly (as per japanese mode):

langen^^!s0^Then, after letting two large tears...^/
langen!d1600^drip down the paper, ^@/
langen^...she flew out of the room, ^/
langen!d1600^hugged Maria from behind as she headed for the bathroom, ^@/
langen^......buried her face into her daughter's back...^/
langen!d3000^and cried.^/
langjp!sd\
langen!sd\

Also, why are all the delay values multiplied by 2 as compared to the japanese lines? Will have to check the git logs to see if someone just doubled all t he delays in the entire script.


I think the following steps are required:

drojf commented 6 years ago

This should be fixed after applying 1543d8c5a80bc25d8df1bb1e7daa5768c14bf4b8, 1b2fa16d2330fb35691571314fc16a2a7bbfb8db, and a19b17a751e378b9c1dafba1534d454fc94b3fb5 :

drojf commented 6 years ago

While I was fixing this issue, I found some voice files present in the script (but not necessarily used in the script) which don't exist on disk:

35 Missing voice files:
voice\16\10600020.ogg
voice\12\20100250_1.ogg
voice\27\20701429.ogg
voice\10\20101212.ogg
voice\10\20101219.ogg
voice\10\20101220.ogg
voice\10\20101265.ogg
voice\01\31500075.ogg
voice\01\31500079.ogg
voice\01\31500080.ogg
voice\01\31500093.ogg
voice\10\30100414.ogg
voice\10\30100657.ogg
voice\03\30900096.ogg
voice\10\30101271.ogg
voice\10\30101469.ogg
voice\10\30101613.ogg
voice\11\32000001.ogg
voice\11\42000026.ogg
voice\11\42000045.ogg
voice\11\42000117.ogg
voice\11\42000277.ogg
voice\11\42000951.ogg
voice\11\42000972.ogg
voice\11\42001053.ogg
voice\13\40401069.ogg
voice\11\42001401.ogg
voice\11\42001463.ogg
voice\10\40100727.ogg
voice\10\40100821.ogg
voice\11\42001671.ogg
voice\11\42001693.ogg
voice\11\42001698.ogg
voice\11\42001746.ogg
voice\10\90100412.ogg

If anyone wants to go through these and check whether they're actually used or not, be my guest (but leave a comment here if you decide to do it)

zeparu commented 6 years ago

there is still (at least one) spot in episode 2, where the text still auto-advances before the voiceline finished playing, and i cant see any voicedelay calls in those lines, its the lines "Aah, I want a boyfriend toooo!! I can't believe you got ahead of me!! Even though we promised that we'd fine boyfriends at the same time, you beat me, aarrrrrrgh!" which are played almost instantly after one another, while jessica's voicelines are skipped over several times.

Something special on those lines is that they dont appear letter for letter on the screen, but every sentence/line is displayed instantly, i don't know if any other of those lines could also still suffer from skipping the voicelines

drojf commented 6 years ago

hmm interesting, in the old umitweak version, the delays are there, but it's missing from our scripts...:

Old Umitweak Script

; 紗音に先を越されたのが悔しー!!/
delay 3150:dwave 0, jes_2e103`I can't believe Shannon got ahead of me!!  `/

se1 7
quakey 5,500

; 彼氏作る時は一緒だぜって約束したのに抜け駆けされたー、/
delay 2750:dwave 0, jes_2e104`Even though we promised that we would find boyfriends at the same time, you beat me, `/
;!d600むがーー!」/
delay 3800:dwave 0, jes_2e105`Aarrrrrrgh!"`/

Our Script

langjp^^!s1「あーッ、私も彼氏ほしー!!/
langen:dwave 0, jes_2e102:^^!s1^"Aah, I want a boyfriend toooo!!^/
se1 6
quakey 3,400
langjp 紗音に先を越されたのが悔しー!!/
langen:dwave 0, jes_2e103:^  I can't believe you got ahead of me!!^/
se1 7
quakey 5,500
langjp 彼氏作る時は一緒だぜって約束したのに抜け駆けされたー、/
langen:dwave 0, jes_2e104:^  Even though we promised that we'd find boyfriends at the same time, you beat me,^/
langjp^^!d600むがーー!」/
langen:dwave 0, jes_2e105:^ aarrrrrrgh!"^/

I'll probably write a script to cross check the delays between the scripts, and insert delays appropriately.

drojf commented 6 years ago

OK, I pushed the changes just now. Quite a few lines were fixed, so it would be worth updating your script file (the script file is the same number of lines, so your saves should be compatible).

zeparu commented 6 years ago

sadly, i just encountered another spot, where the text automatically goes on, cutting of voice lines in the process, there are no mentions of any voicedelay in those lines

specifically its

se1 3
langjp「うっそー、ヒナの彼氏ィ超ォイケてんじゃん〜ッ!!」/
langen:dwave 0, "voice\99\frefa2001.ogg":^"No wa-y, Hina's boyfriend's so a-wesome!!"^/
langjp!d1000
langen!d1000

se2 4
langjp「マジすごくない?!/
langen:dwave 0, "voice\99\frefb2001.ogg":^  "Isn't he?!^/
langjp!d800/
langen!d800/

se3 4
langjp マジすごくない?!」/
langen:dwave 0, "voice\99\frefb2002.ogg":^  Isn't he?!"^/
langjp!d700
langen!d700

se1 5
langjp「リンの彼氏すっごい頭良さそォ!/
langen:dwave 0, "voice\99\frefc2001.ogg":^  "Rin's boyfriend looks re-ally smart!^/
langjp!d800/
langen!d800/

se2 5
langjp あれで鬼畜眼鏡なの、マジ本当ォ?!」/
langen:dwave 0, "voice\99\frefc2002.ogg":^  Is it true that guy's a glasses beast?!"^/
langjp!d900
langen!d900

se3 9
langjp「ダメダメ、うちのダンナ、ギャル男だしー!」/
langen:dwave 0, "voice\99\frefb2003.ogg":^  "No way, no way, my husband'll be a pretty bo-y!"^/
langjp!d1000
langen!d1000

se1 3
langjp「「「きゃっきゃッ♪ きゃっきゃッ♪」」」/
langen:dwave 0, "voice\99\alcmix0003.ogg":^  """*giggle*♪ *giggle*♪"""^/
langjp!sd\
langen!sd\

at line 92445 onwards

i dont know if there are any others still left, but by the fact that i still found some, chances are there are other places still where this will happen again

drojf commented 6 years ago

The part

langjp!d1000
langen!d1000

is supposed to add a delay, but clearly isn't working. I'll check it out.

EDIT: actually I can see what happened - because it used a different delay method, my script must have missed it, so didn't adjust the delay values to match the voice file length...

There are actually a few different wait commands:

delay [delaytime]
wait [delayitme]

and the short forms respectively:

!d[delaytime]
!w[delaytime]

Thanks for reporting the bugs man

zeparu commented 6 years ago

ok, with these commits the part that i specifically pointed out is perfectly fine now, but i already found the next troublesome spot, at line 93528 onwards

langjp^^!s1「違うよぉ、あの子だよ、ジェシの彼氏ぃ!!」/
langjp^^!d600「ウッソ?!/
langjp^^!d300 あの子がぁ?!」/
langjp^^!d500「信じらんないよね、絶対にエアだと思ってたよォ!」/
langjp^^!d600「可愛いじゃーん、年下ァ?!/
langjp^^!d300 いいなぁ!!」/
langen:dwave 0, "voice\99\stufb2001.ogg":^^!s1^"No, it's that kid, that's Jessie's boyfriend?!!" ^/
langen:voicedelay 1090:dwave 0, "voice\99\stufa2004.ogg":^"No way?!^/
langen:voicedelay 600:dwave 0, "voice\99\stufa2005.ogg":^  It's that kid?!" ^/
langen:voicedelay 1000:dwave 0, "voice\99\stufc2001.ogg":^"I can't believe it, I was sure he was fake!" ^/
langen:voicedelay 1200:dwave 0, "voice\99\stufa2006.ogg":^"Isn't he cu-te?  So he's younger?!^/
langen:voicedelay 600:dwave 0, "voice\99\stufa2007.ogg":^  Lucky Jessie!!"^/
langjp!sd\
langen!sd\

seems like they just have placeholder values and haven't been properly set yet, i don't know if you are still in the process of updating these, but i thought i'd just point them out just in case

drojf commented 6 years ago

I don't think I would have caught those - I think my original script expected the format

dwave 0, jes_2e103

but sometimes they use the raw voice file

dwave 0, "voice\99\stufa2004.ogg"

I'll go ahead and try to fix up those too.

drojf commented 6 years ago

I fixed it up, but I don't suggest you look at the actual commit log because it might have spoilers.

The non-spoiler fixes are below:

----------------------------
langen:dwave 0, "voice\99\stufb2001.ogg":^^!s1^"No, it's that kid, that's Jessie's boyfriend?!!" ^/
-langen:voicedelay 1090:dwave 0, "voice\99\stufa2004.ogg":^"No way?!^/
-langen:voicedelay 600:dwave 0, "voice\99\stufa2005.ogg":^  It's that kid?!" ^/
-langen:voicedelay 1000:dwave 0, "voice\99\stufc2001.ogg":^"I can't believe it, I was sure he was fake!" ^/
-langen:voicedelay 1200:dwave 0, "voice\99\stufa2006.ogg":^"Isn't he cu-te?  So he's younger?!^/
-langen:voicedelay 600:dwave 0, "voice\99\stufa2007.ogg":^  Lucky Jessie!!"^/
+langen:voicedelay 3180:dwave 0, "voice\99\stufa2004.ogg":^"No way?!^/
+langen:voicedelay 650:dwave 0, "voice\99\stufa2005.ogg":^  It's that kid?!" ^/
+langen:voicedelay 870:dwave 0, "voice\99\stufc2001.ogg":^"I can't believe it, I was sure he was fake!" ^/
+langen:voicedelay 3860:dwave 0, "voice\99\stufa2006.ogg":^"Isn't he cu-te?  So he's younger?!^/
+langen:voicedelay 1850:dwave 0, "voice\99\stufa2007.ogg":^  Lucky Jessie!!"^/

----------------------------

langen:dwave 0, "voice\99\awase0011.ogg":^^!s1^"""Jessie-samaaaaaaaa!!^/
 se2 2
 langjp^^!d600 ジェシィイジェシィイイイイィ!!!」」」/
-langen:voicedelay 1200:dwave 0, "voice\99\awase0012.ogg":^  Jessiiiiie, Jessiiiiiiiiiiie!!!"""^/
+langen:voicedelay 1680:dwave 0, "voice\99\awase0012.ogg":^  Jessiiiiie, Jessiiiiiiiiiiie!!!"""^/

----------------------------

 langen:dwave 0, "voice\99\stufa4004.ogg":^^!s1^"Huh?!^/
-langen:voicedelay 4440:dwave 0, "voice\99\stufa4005.ogg":^  Is someone there?!"^/
+langen:voicedelay 390:dwave 0, "voice\99\stufa4005.ogg":^  Is someone there?!"^/

I'll leave this thread open - the number of errors found indicates I probably havent fixed everything.

drojf commented 5 years ago

Haven't had any reports about this for a while, so I'll assume it's fixed.

zeparu commented 5 years ago

found one more at line 217513 onwards

langjp:dwave_jp 0, rg1_4e30:「“下水道”、/
langen:dwave_eng 0, rg1_4e30:^"`Gesuidou' (Drainage system), ^/
se1 47
wait 300
langjp:dwave_jp 0, rg1_4e31:“埋立地”、/
langen:dwave_eng 0, rg1_4e31:^`umetatechi' (reclaimed land), ^/
se2 48
wait 300
langjp:dwave_jp 0, rg1_4e32:“地下室”。/
langen:dwave_eng 0, rg1_4e32:^`chikashitsu' (basement).^/

delay constantly at 300

drojf commented 5 years ago

There were about 6 points in the script with the same problem (some cases would be unnoticeable though), which should be fixed now. I'll leave the issue open for now.

Also not sure if I have to fix this for the answer arcs too

edit: reminder to self that the scripts for fixing are in the Fix_Automatic_Voice_Lines folder of this repo: https://github.com/drojf/umineko_python_scripts