Welcome to Doom9's Forum, THE in-place to be for everyone interested in DVD conversion. Before you start posting please read the forum rules. By posting to this forum you agree to abide by the rules. |
|
|
Thread Tools | Search this Thread | Display Modes |
17th March 2009, 00:01 | #82 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
16.03.2009 1.9 -> 2.0
Last edited by 0xdeadbeef; 17th March 2009 at 00:16. |
17th March 2009, 02:37 | #83 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
Regarding the MKVMerge issue, I got a little more insight what causes this problem. Indeed MKVMerge doesn't complain about a time overlap, but about a "wrong" offset. This seems to happen for each subtitle that doesn't fit into one 0x800 bytes packet and thus uses an RLE buffer split over multiple (usually 2) packets.
While I'm not so keen to claim that there is no chance that I messed something in my implementation, chances are that MKVMerge doesn't consider multi-packets at all. At least the fact that VobSync is able to decode the multi-packet subtitles created by BDSup2Sub hints that the command buffer and the offset can't be that wrong. Besides, I'm pretty sure I saw multi-packet subtitles displayed correctly when used to create a DVD. So this might be an issue of MKVMerge. I'm not yet 100% sure though. |
17th March 2009, 06:04 | #85 | Link |
Registered User
Join Date: May 2008
Posts: 1,840
|
Thanks for the update and uniterrupted download of it.
I tried the wildcard cli: java -jar BDSup2Sub.jar "'00001*.sup' '*-bbsup2sub.sup /res:720" and I got Code:
OPTION: set resolution to 720p (1280x720) Exception in thread "main" java.lang.NullPointerException at BDSup2Sub.main(BDSup2Sub.java:313) Also exported sups (1080 and 720) are displaying correctly now in mpc-hc and powerdvd, great work! Last edited by turbojet; 17th March 2009 at 06:12. |
17th March 2009, 12:24 | #86 | Link | |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
Quote:
Code:
java -jar BDSup2Sub.jar "00001*.sup *-bbsup2sub.sup /res:720" |
|
17th March 2009, 15:39 | #87 | Link |
Registered User
Join Date: May 2008
Posts: 1,840
|
With same files but no space in path I tried: java -jar BDSup2Sub.jar "00001*.sup *-bbsup2sub.sup /res:720"
and the same error popped up I also tried: java -jar BDSup2Sub.jar "00001*.sup *-bbsup2sub.sup" /res:720 and got: ERROR: No extension given for target /res:720 |
17th March 2009, 18:07 | #88 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
Ok, it has nothing to do with single quotes. It's just that I always tested with the source files in a path. So I overlooked that no path inside the source file name results in a null pointer exception.
Until I fix this in the next release, use the following workaround: Code:
java -jar BDSup2Sub.jar ".\00001*.sup *-bbsup2sub.sup /res:720" |
17th March 2009, 19:56 | #90 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
While we're at it, there are some restrictions to the characters that the wildcard will replace. Indeed more or less only alphanumeric characters (and ".") are supported. I guess I will switch to a more relaxed regular expression for the next release, bur for the moment, special characters like "(){}[]!%&§#~" etc. are not matched by the wildcard.
I also made some progress regarding the MKVMerge issue. Indeed it's really a bug in BDSup2Sub. It happens only for subtitle frames where the RLE data would fit into 0x800 bytes, but due to the control header, a 2nd packet is needed. In this case, the overall packet length is wrong which is exactly what MKVMerge reports (if you know how to take it). VobSub Resync seems to be pretty relaxed about this and SubtitleCreator crashes when it tries to show such a subtitle. I have also at least one report that hints that this could lead to problems with standalone DVD players. -> Will be fixed in next version. Last edited by 0xdeadbeef; 17th March 2009 at 20:31. |
17th March 2009, 21:36 | #91 | Link | |
Matroska find' ich toll
Join Date: Apr 2008
Posts: 1,370
|
Quote:
I had converted some BD.sups and found another issue. in the idx-file i found some entries like this: timestamp: 01:26:34:439, filepos: 000ce2000 timestamp: 00:00:00:000, filepos: 000ce3800 timestamp: 01:26:39:444, filepos: 000ce6800 timestamp: 01:26:40:486, filepos: 000ceb800 thats the last 4 entries! in another case: timestamp: 01:26:37:775, filepos: 000cdc000 timestamp: 01:26:41:613, filepos: 000cdf000 timestamp: 00:00:00:000, filepos: 000ce2000 timestamp: 00:00:00:000, filepos: 000ce4000 i think this entries are to much. hubble sample Last edited by hubblec4; 18th March 2009 at 06:00. |
|
17th March 2009, 22:25 | #92 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
Approval may take a while, better use Sendspace or whatever. Then again I had enough for today anyway.
17.03.2009 2.0 -> 2.1
|
18th March 2009, 06:14 | #94 | Link | |
Matroska find' ich toll
Join Date: Apr 2008
Posts: 1,370
|
Quote:
but the 00:00:00-timestamp-entries are there. thats the last entries in the idx-file(BDSup2sub v.2.1) timestamp: 01:26:34:355, filepos: 000ccb800 timestamp: 00:00:00:000, filepos: 000cce800 timestamp: 00:00:00:000, filepos: 000cd1000 timestamp: 01:26:41:446, filepos: 000cd6000 timestamp: 01:26:41:613, filepos: 000cd8800 timestamp: 00:00:00:000, filepos: 000cdb000 timestamp: 00:00:00:000, filepos: 000cdc800 its possible to delete this entries automaticly? sample hubble Last edited by hubblec4; 18th March 2009 at 06:18. |
|
18th March 2009, 12:19 | #95 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
Invalid timestamp are usually corrected by BDSup2Sub. There are obviously special circumstances here that I need to examine.
It's also possible that I broke this correction (partly) in 2.0 or 2.1. It basically still works though, as I tested several SUPs with invalid time stamps before I released 2.1. We'll see when I get back from work and can have a look at your samples. [Edit] Ok, let's say it will be fixed in the next release Last edited by 0xdeadbeef; 18th March 2009 at 19:12. |
19th March 2009, 08:16 | #97 | Link | |
Matroska find' ich toll
Join Date: Apr 2008
Posts: 1,370
|
Quote:
question: sometimes is the time too short for showing the subtitles. is it possible manuelly change the start- and endtime in your tool? :-) hubble |
|
19th March 2009, 08:31 | #98 | Link |
Registered User
Join Date: Sep 2003
Location: On The Beach
Posts: 714
|
I don't think that is possible. But maybe to add some check if a subtitle have duration under 1 second (23 frames), to automatically extend the duration to at least 23 frames for any subtitle, but with the caution to not overlap the timmings with the next one.
enjoy, Mtz |
19th March 2009, 12:19 | #100 | Link |
Author of BDSup2Sub
Join Date: Jun 2003
Posts: 478
|
Editing the times/position is not really planned currently as it would involve a redesign of several program parts.
Detecting too short subtitles is something I thought about some time ago but simply forgot. I'd like to output a warning as default and fix the length only if it's selected via (export) menu. Last edited by 0xdeadbeef; 19th March 2009 at 12:22. |
Thread Tools | Search this Thread |
Display Modes | |
|
|