-
Notifications
You must be signed in to change notification settings - Fork 13
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
HandBrake converts all subtitles to ASS vs. passthrough #95
Comments
ptr727
added a commit
that referenced
this issue
Mar 23, 2022
- Version 2.6: - Fixed `SidecarFile.Update()` bug that would not update the sidecar when only the `State` changed, and kept re-verifying the same verified files. - Added a `--reprocess` option to the `process` command, `process --reprocess [0 (default), 1, 2]` - The `--reprocess` option can be used to override conditional sidecar state optimizations, e.g. don't verify if already verified. - 0: Default behavior, do not do any reprocessing. - 1: Re-process low cost operations, e.g. tag detection, closed caption detection, etc. - 2: Re-process all operations including expensive operations, e.g. deinterlace detection, bitrate calculation, stream verification, etc. - Whenever processing logic is updated or improved (e.g. this release), it is recommended to run with `--reprocess 1` at least once. - Added workaround for HandBrake that [force converts](HandBrake/HandBrake#160) closed captions and subtitle tracks to `ASS` format. - After HandBrake deinterlacing, the original subtitles are added to the output file, bypassing HandBrake subtle logic. - Subtitle track formats and attributes are preserved, and closed captions embedded are not converted to subtitle tracks. - The HandBrake issue tracked as [#95](#95). - Added the removal of [EIA-608](https://en.wikipedia.org/wiki/EIA-608) Closed Captions from video streams. - Closed Caption subtitles in video streams are undesired as they cannot be managed, all subtitles should be in discrete tracks. - FFprobe [fails](https://www.mail-archive.com/[email protected]/msg126211.html) to set the `closed_captions` JSON attribute in JSON output mode, but does detect and print `Closed Captions` in normal output mode. - FFprobe issue tracked as [#94](#94). - Added the ability to bootstrap 7-Zip downloads on Windows, manually downloading `7za.exe` is no longer required. - Getting started is now easier, just run: - `PlexCleaner.exe --settingsfile PlexCleaner.json defaultsettings` - `PlexCleaner.exe --settingsfile PlexCleaner.json checkfornewtools` - The `--mediafiles` option no longer supports multiple entries per option, use multiple `--mediafiles` options instead. - Deprecation warning initially issued with v2.3.5. - Old style: `--mediafiles path1 path2` - New style: `--mediafiles path1 --mediafiles path2` - Improved the metadata, tag, and attachment detection and cleanup logic. - FFprobe container and track tags are now evaluated for unwanted metadata. - Attachments are now deleted before processing, eliminating problems with cover art being detected as video tracks, or FFMpeg converting covert art into video tracks. - Run with `process --reprocess 1` at least once to re-evaluate conditions. - Removed the `upgradesidecar` command. - Sidecar schemas are automatically upgraded since v2.5. - Removed the `verify` command. - Use `process --reprocess 2` instead. - Removed the `getbitrateinfo` command. - Use `process --reprocess 2` instead. - Minor code cleanup and improvements.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
HandBrake will convert closed captions and subtitle tracks to ASS format.
The behavior is "intentional", but undesirable.
https://forum.handbrake.fr/viewtopic.php?f=6&t=34522&p=161536
Issue is tracked here.
HandBrake/HandBrake#160
As workaround run handbrake with no subtitles in output, and then remux subtitles back in.
The text was updated successfully, but these errors were encountered: