cbusillo / BD_to_AVP

3D Blu-ray to Apple Vision Pro converter
GNU General Public License v3.0
99 stars 5 forks source link

No SRT files created message #21

Closed edeuss closed 5 months ago

edeuss commented 5 months ago

Seem to get a no SRT files created when it does create the files and the their are subtitles.

Screenshot 2024-06-18 at 12 29 39 PM Screenshot 2024-06-18 at 12 30 04 PM

Using GUI V0.2.108

cbusillo commented 5 months ago

Hi! What size are the files. Are they zero bytes?

cbusillo commented 5 months ago

If they are zero bytes the OCR library is returning no subs. I have opened an issue with the author of that library. Subs are complicated especially from BD3D. They are in an image format which isn't used anymore. I don't believe I can convert to an image format that MV-HEVC will use, so I have to convert them to text with the pgsrip library. I am open to other solutions!

edeuss commented 5 months ago

@cbusillo They have data in them.

Screenshot 2024-06-19 at 12 08 34 AM
cbusillo commented 5 months ago

Odd, so it's a different issue. Are there any non ascii characters in the filename? Just a FYI, I think checking continue on error should allow it to continue, and since they are valid they should be added to the output. It should be seeing those files and NOT give you an error.

if not any(output_path.glob("*.srt")) and not skip_or_continue:
    raise SRTCreationError("No SRT subtitle files created.")
edeuss commented 5 months ago

@cbusillo This is the file name Pirates.of.the.Caribbean.Dead.Men.Tell.No.Tales.2017.mkv

cbusillo commented 5 months ago

I cannot figure out how you would get this error while there are srt files with data in them. Have you tried it a second time to see if it was a transient issue? Did you try with "Continue Processing on Error" to see if the subs wind up in the final video? Have you been able to successfully encode other videos with this version? Thank you!

mikemonello commented 5 months ago

I am getting the same error on some ISO files that have subtitles. Normally not a big deal, but this one is a French film so English subtitles are a must for me, hahaha

cbusillo commented 5 months ago

@mikemonello are there srt files? if so how large are they?

edeuss commented 5 months ago

@cbusillo I have tried two movies with the same effect, the video encodes correctly just with no subtitles even though it successfully extracts them in the step before.

cbusillo commented 5 months ago

@edeuss are you ripping to and from the internal drive, an external drive, or a network drive? Also I didn't notice you answering if the sups are in the final video when you select continue on error. Everything looks right in the code and it works for me. I cant think of any reason it wouldn't find the *.srt files in the output_folder. Can I have a screen shot of the full app window so I can see all of the settings, just in case that will help me?

cbusillo commented 5 months ago

Adding a new prerelease, it should be ready in 20 minutes. It changes the detection method, but also adds a bit of debugging text.

Running Sup subtitles extraction and SRT conversion 
Checking for .srt files in: /Users/cbusillo/TEMP/Blu-Ray

Found .srt file: title_t00.en.srt

Finished Sup subtitles extraction and SRT conversion in 57 seconds

Running FRIM to split MVC to stereo. 🌖
edeuss commented 5 months ago

@edeuss are you ripping to and from the internal drive, an external drive, or a network drive?

From an internal drive through makemkv.

Also I didn't notice you answering if the sups are in the final video when you select continue on error.

Thin this comment I said that once continuing the video encoded fine but there was no subtitle attached.

Can I have a screen shot of the full app window so I can see all of the settings, just in case that will help me?

Screenshot 2024-06-21 at 9 50 09 AM

Just using the default settings.

edeuss commented 5 months ago

@cbusillo I think I do have an idea of what the issue is though. When its going though the process and it creates a temp folder, these .srt's are not in that. They are in the same folder as the source .mkv, so maybe it's looking for them in the temp folder instead of where they are.

cbusillo commented 5 months ago

@edeuss, thank you, I misinterpreted that message. I notice this one has symbols in the filename, that shouldn't matter and you also had issues with a "normal" filename.

It is (should be) looking in the correct place, the temp folder was to move all of the temporary files out of the main drive, just in case they were large. The issue did seem to start around that time though, so I'll keep looking at it.

Did you happen to update to at least prerelease v0.2.113? I added some path information in the output so we can figure this out.

mikemonello commented 5 months ago

I received the following error using v0.2.113. For this one, I ripped both 3DBDs to MKV using MakeMKV, then merged the files into 1 3D MKV. I then used the app to convert that file, but I did verify the subtitles worked in VLC before running the conversion.

🟢 Processing started at 11:49:53 PM on 2024-06-20 🟢 Processing /Volumes/Movespeed2TB_SSD/MakeMKVTests/Titanic3D_merged.mkv

Running Get disc and MVC video properties Finished Get disc and MVC video properties in a moment

Detecting crop parameters...

Running ffmpeg to extract video, audio, and subtitles from MKV Finished ffmpeg to extract video, audio, and subtitles from MKV in 16 minutes

Running Sup subtitles extraction and SRT conversion Checking for .srt files in: /Volumes/3DAVP_Movies/BD_To_AVP_GUI_OUTPUT/Titanic3D_merged

No .srt files found in /Volumes/3DAVP_Movies/BD_To_AVP_GUI_OUTPUT/Titanic3D_merged

No SRT subtitle files created. ❌ Processing failed in an hour ❌ 🛑 Processing stopped after an hour 🛑

cbusillo commented 5 months ago

@mikemonello I didn’t notice an answer earlier, are those files bigger than 0 bytes? I’m trying to make sure this is the same issue. I’m not sure why I am not having the same issue. Can you give a screenshot of the Titanic3D_merged folder? Also just out of curiosity @edeuss what are some reasons you rip to MKV first? I understand Mike’s reason here, since there are two discs.

mikemonello commented 5 months ago

I'm going to install vo.2.114 and try again, but here's a screenshot of the folder. No .srt files were there, and the tmp folder is empty.

Screenshot 2024-06-21 at 10 12 06 AM
mikemonello commented 5 months ago

Just found them! I went to run a new rip and the .srt files were in the folder where the source MKV file was, not in the output folder.

Screenshot 2024-06-21 at 10 17 54 AM

Both files have data in them:

Screenshot 2024-06-21 at 10 19 54 AM Screenshot 2024-06-21 at 10 20 07 AM
cbusillo commented 5 months ago

I misread @edeuss's statement above. Thank you @mikemonello, I will figure out what's going on now that I can see it.

cbusillo commented 5 months ago

You two are using mkvs as sources. I think that may be causing the issue. I always start with an iso.

cbusillo commented 5 months ago

Sorry for not understanding! MKVs are handled slightly differently since they are basically a stage two, but we need some info from stage one. pgsrip doesn't allow me to specify an output folder, it outputs in the source folder. For ISOs thats fine since the mkv is created in the working folder. I should be able to push a fix out for this shortly.

mikemonello commented 5 months ago

I did select the option to start on step 2 after the MKV rip, since I had already done that.

cbusillo commented 5 months ago

I think v0.2.115 will fix it when it comes out. If anyone has time to test on a MKV, please let me know. That was the issue. Also @mikemonello FYI, you can use stage 1 or 2. I basically does the same thing if the input is a MKV. Thank you both for helping me thought this one.

edeuss commented 5 months ago

@cbusillo Seems to work now just the subtitle names are named oddly.

Screenshot 2024-06-21 at 8 10 21 PM
cbusillo commented 5 months ago

@edeuss what would you expect them to be named? Do you have examples of ones you think are named correctly?

mikemonello commented 5 months ago

I keep getting this same hangup. The icon isn't animating, and it's been at this stage for almost 2 hours. The app hasn't crashed, it's responsive to the system, it just seems stuck.

Screenshot 2024-06-21 at 6 23 45 PM
cbusillo commented 5 months ago

You keep getting it as in you are getting it on just this movie, or multiple movies? Can you should me the Titanic 3D folder?

cbusillo commented 5 months ago

I noticed this is a video you "merged". does the same thing happen on the individual ISOs? @mikemonello?

cbusillo commented 5 months ago

This should be fixed, MKVs as source have the srt files created in the source folder instead of destination. Moving them to output folder fixed issue