ttml2ssa icon indicating copy to clipboard operation
ttml2ssa copied to clipboard

Output .ssa files are malformed according to Aegisub

Open flagg19 opened this issue 3 years ago • 3 comments

Hi there, I'm trying to edit the .ssa file produced by your tool, using Aegisub. It says the file is malformed:

image

I also tried to load (without modification) the produced file in MPC-HC and if fails with no error.

Using a random online tool to do the conversion, this is what I've found:

yours

[V4+ Styles]
Format: Name, Fontname, Fontsize, PrimaryColour, BackColour, OutlineColour, Bold, Italic, Alignment, BorderStyle, Outline, Shadow, MarginL, MarginR, MarginV
Style: Default,Arial,50,&H00FFFFFF,&H40000000,&H00000000,0,0,2,1,2,3,40,40,40

[Events]
Format: Layer, Start, End, Style, Text
Dialogue: 0,0:00:06.31,0:00:07.52,Default,{\i1}This doesn't work{\i0}

another online tool

[V4+ Styles]
Format: Name, Fontname, Fontsize, PrimaryColour, SecondaryColour, OutlineColour, BackColour, Bold, Italic, Underline, StrikeOut, ScaleX, ScaleY, Spacing, Angle, BorderStyle, Outline, Shadow, Alignment, MarginL, MarginR, MarginV, Encoding
Style: Default,monospace,20,&H00FFFFFF,&H0300FFFF,&H00000000,&H02000000,0,0,0,0,100,100,0,0,1,2,2,2,10,10,10,1

[Events]
Format: Layer, Start, End, Style, Actor, MarginL, MarginR, MarginV, Effect, Text
Dialogue: 0,0:00:06.31,0:00:07.52,Default,,0,0,0,,{\i1}This works{\i0}

Other than minor differences in the style, the main difference is that yours lacks some format fields. It makes sense given the error message "not enought fields".

I'm no subtitle expert, I don't know what's the minimum set of fields required, here there's a good description of what they do but no mention if they are mandatory or not: https://forum.videohelp.com/attachment.php?attachmentid=33290&d=1440307546

Anyway, I'm forking (https://github.com/Paco8/ttml2ssa/compare/main...flagg19:ttml2ssa:main) and patching it up adding all missing fields with an empty/default value just to make it work because I need it. I'm opening this issue in case you didn't know and want to do a proper fix.

flagg19 avatar Jul 26 '22 16:07 flagg19

great to find a fixing fork, but there is no working build as the release version is still 0.3.7, would you kindly release a version that fixes the problem?

YuenSzeHong avatar Aug 12 '22 11:08 YuenSzeHong

great to find a fixing fork, but there is no working build as the release version is still 0.3.7, would you kindly release a version that fixes the problem?

here you go: https://github.com/flagg19/ttml2ssa/releases/tag/v0.3.7-fork

I haven't changed the version number because I don't know what's the proper etiquette for versions in forks

flagg19 avatar Aug 12 '22 13:08 flagg19

thanks

YuenSzeHong avatar Aug 16 '22 02:08 YuenSzeHong

This should be fixed in version 0.3.8.

Paco8 avatar Dec 07 '22 14:12 Paco8