mirror of
https://github.com/Radarr/Radarr.git
synced 2026-03-02 22:57:34 -05:00
Subtitle custom format support #8249
Labels
No labels
Area: API
Area: Database
Area: Db-migration
Area: Download Clients
Area: Extras
Area: Import Lists
Area: Indexer
Area: Metadata API
Area: Notifications
Area: Organizer
Area: Parser
Area: Scanning
Area: Tooling
Area: UI
Area: Unit Tests
On Hold: MetadataAPI Blocking
On Hold: MetadataAPI Blocking
Priority: High
Priority: Low
Priority: Medium
Status: Accepted
Status: Cannot Reproduce
Status: Confirmed
Status: Help Wanted
Status: In Progress
Status: Indexer - need invite
Status: Info Needed
Status: Investigating
Status: Logs Needed
Status: Maybe One Day
Status: Needs Triage
Status: On Hold
Status: Ready for Review
Status: Unlikely
Status: Waiting for OP
Status: Won't Fix
Type: Bug
Type: Documentation
Type: Duplicate
Type: Enhancement
Type: External Bug
Type: Feature Request
Type: Regression
Type: Support
Type: Support.
conflict
lidarr-pull
no-conflict
not-pulled
readarr-pull
readarr-pull
sonarr upstream
sonarr-pull
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
starred/Radarr#8249
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @Maxmystere on GitHub (Jan 28, 2024).
Is there an existing issue for this?
Is your feature request related to a problem? Please describe
I would like to be able to do the same as the current LanguageSpecification but for subtitles
Describe the solution you'd like
Adding a new
CustomFormatSpecificationBasethat would read theinput.Movie.MovieFile.MediaInfo.SubtitlesDescribe alternatives you've considered
Using the Release Title condition but it isn't really practical
Anything else?
.
@Maxmystere commented on GitHub (Jan 28, 2024):
A practical usage would be to ensure the file has the subtitles embedded (for example mkv file)
@Qstick commented on GitHub (Feb 4, 2024):
The problem is, we won't know what it is 50% of the time prior to downloading. So upgrades will never get grabbed reliably.
@Maxmystere commented on GitHub (Feb 4, 2024):
From what I saw this is what the Language Custom Format already does no ?
@Qstick commented on GitHub (Feb 4, 2024):
Language is in the release title much more often than subtitle language is
@Maxmystere commented on GitHub (Feb 4, 2024):
Don't hesitate to tell me if I'm wrong with anything I say about the code
After checking a bit more the LanguageSpecification class
github.com/Radarr/Radarr@4a9c0b2240/src/NzbDrone.Core/CustomFormats/Specifications/LanguageSpecification.cs (L23)It's detecting the language before downloading then also after with the resulting downloaded file
Before:
github.com/Radarr/Radarr@4a9c0b2240/src/NzbDrone.Core/CustomFormats/CustomFormatCalculationService.cs (L44)After:
github.com/Radarr/Radarr@4a9c0b2240/src/NzbDrone.Core/CustomFormats/CustomFormatCalculationService.cs (L207)So if I understood well the issue with my proposal is that there is no "before" checkup ?
My goal was to improve the already existing basic subtitle detection possible with trash-guide and make it possible to invalidate a downloaded files to try to find a better match if no subtitles were found
@Qstick commented on GitHub (Feb 4, 2024):
Correct, very rarely do we know the subtitle language BEFORE download. This results in an issue that the file on disk will always be scored higher than any release on the indexer (simply due to us not knowing the sub languages). It's hard to implement a custom format specification on a property where the BEFORE and AFTER are not each reliable.
@Maxmystere commented on GitHub (Feb 4, 2024):
My idea for using this Custom format was to give a negative score if no good subtitles were found
For example it would give a score of 0 before downloading then after downloading would either give a +1 or -1 to the resulting file
Is this a relevant use case for the Custom Formats ?