-
Notifications
You must be signed in to change notification settings - Fork 11
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
Editing of one enrty cannot be successful what so ever #16
Comments
The only difference between the two files you provided is the text content and length (if they are both utf-8 encoded). The "corrupted" one seems to be missing the character ID prefixes ([C=37]) in each line. Perhaps the game can only "skip" to lines with a character ID. Original:
Modded:
|
I already tried having the character ID prefixes, but same result. |
Entry Id="856784307" Original https://mega.nz/#!uRVTSDTa!UB7xpGYQY5WcEQrw02G0JF2MqC1-YX9YZPiCWPUkKtg Look at Lines# 138 & 336 in the Modified version, the file works probably, that's because I didn't touch these 2 lines.. |
on the last sample that I provide, I did some tests. |
It could also be related to the characters in a line and the line length. Your example is a lot larger than the original line. Could you try replacing the original line with substrings of varying length of your modded line? |
Yes I tried that already, I even put english words instead -with respect of crossing the maximum length-, l am pretty sure that the problem happens because I passed the maximum length in the entry. |
The conclusion of this, is that whenever the -length- if the whole enrty exceeds a specific lenght that each entry could take, no matter what substring is causing that, substring itself is not related directly but the length of the whole thing. Hope if there's any means to increase the "length limit". |
You're right with the different sizes for different UTF-8 codepoints. As each entry in a subp file is saved as a single string with $-characters separating the lines, the max character limit per entry should be (assuming the entry has at least one line):
As the game can't load these files correctly there have to be some other limits. |
update: seems like that reducing the length in the first sample cannot help either. As for flags that you mentioned, I reported earlier on another raised issue that Flags value is related to content, |
Combining 2 lines will just save a single byte. Did you check if the size limit you found is the same for each subp file or if some of them have different limits? |
You're mapping arbitrary Japanese UTF-8 codepoints to Arabic letters, right? Could you try using only codepoints that are 1 byte wide instead of using the 3 byte ones? |
I had a third entry sample also which was corrupted but know I managed to fix it. If you want to look at it just let me know, also the 2nd sample is fixed just by removing some unwanted spaces, but the first one is something that cannot be repair I tried using 1 byte letters as you suggested for the first sample, but still they can't cover all of the Arabic letters then I ended up using 2 byte letters with them, still the file is in corrupted status. Even if I merged line texts -which was the solution for the third sample- still no benefit. I managed to shrink to length to 21862 with 2,3 byte letters, and to 17,537 with merging line texts |
How many distinct letters are there in the Arabic alphabet (+numerics and punctuation)? |
Only the letters and punctuation 140 in total, numbers and symbols are shared with Latin, also I cannot replace the one byte Latin letters as I use them almost. |
That's unfortunate. Since I can't change the limits imposed by the engine I'd rather print an error if one of the subtitles doesn't fit in an entry. I'll have to analyze all the unedited subp files to get some more facts about the limits. |
Any information I can provide for this? You just have to ask. And thanks hundred times for the awesome tool |
Could you perhaps upload a zip archive with all subtitles? I don't have the game installed right now and would have to redownload it first. Add me on Steam as sharing all these files publically here is likely against the Github ToS. |
All right, Steam id same as here? |
There are three users by your name, I cannot identify you :) |
Invitation sent |
The entry with id 2161021477 is indeed the largest one in all subs. The max sizes (in bytes) in the unmodified files are as follows: As long as these aren't exceeded the game should load them fine. Anything above these values needs some more testing. |
I will look at this in the evening, thanks. |
I got
|
I'm still continue with my project, and I develop a pattern for localizing the TPP to arabic, which is appear to be successful in every .subp and all entries in them.
But there's one entry that whatever I do it is get corrupted inside the game,
Entry Id="2161021477" in the tape.subp
Cassette tape is Skull Face's Objective [4]
Track Secret Recording of Skull Face and Code Talker [2]
Whenever I modify this entry and put it on the game, the subtitles won't show, and also the rewind and forward buttons are getting corrupted.
I provide a two .subp sample containing just the subjected entry, for both the original and modified one.
Also see the videos to look at the original behavior and the corrupted
Videos
original https://www.youtube.com/watch?v=VpmW2LG6oBk
corrupted https://www.youtube.com/watch?v=xNhw1nteyLg
Samples
Original https://mega.nz/#!zM8khIwb!L-ORh-oHNcA3H1NqgC7YUOijx6oeTvp4BelGiJW6MbU
corrupted https://mega.nz/#!rEkFDSaL!OpJ2ntxyVx3ittkq2r6i72V2U_GoRYIU7LI5YBhx24E
The text was updated successfully, but these errors were encountered: