Skip to content
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

feat(guide-sync): Add more groups for the 3rd party sync applications #2160

Open
wants to merge 27 commits into
base: master
Choose a base branch
from

Conversation

TRaSH-
Copy link
Contributor

@TRaSH- TRaSH- commented Dec 15, 2024

Pull Request

Purpose

Add more groups for the 3rd party sync applications so the end user can easily add/choose certain groups to their QP Sync (Sync
a complete Quality Profile from the guide with all the settings like Custom Formats, Quality Source, Minimum Custom Format Score, Upgrade Until Custom Format Score, etc)

Approach

  • Add group [HDR formats] Dolby Vision Incompatible Devices | (add DV (WEBDL) to your QP)
  • Add group [HDR Formats] DV HDR10+ and HDR10+ Boost | (add the DV HDR10+ and HDR10+ Boost to your QP)
  • Add group [Optional] Miscellaneous | (Choose any CF from the Miscellaneous column)
  • Add group [Optional] Movie Versions | (Choose any Movie version you want)
  • Add group [Optional] SDR | (Choose if you want to use SDR or SDR (no WEBDL))
  • Add group [Required] Golden Rule | (Choose if you want to use x265 (HD) or x265 (no HDR/DV))
  • Add group [Streaming Services] Anime
  • Add group [Streaming Services] Asian
  • Add group [Streaming Services] Dutch
  • Add group [Streaming Services] French
  • Add group [Streaming Services] Miscellaneous
  • Add group [Streaming Services] UK
  • Add group French Source Groups | (Remux Tiers, UHD Bluray Tiers, HD Bluray Tiers and WEB Tiers)
  • Add group German Source Groups | (Remux Tiers, Bluray Tiers and WEB Tiers)
  • Add group HQ Release Groups | (Remux Tiers, UHD Bluray Tiers, HD Bluray Tiers and WEB Tiers)

Open Questions and Pre-Merge TODOs

  • Remove HDR10+ boost and DV (WEBDL) from the Misc UHD group
  • Check group names so they are better sorted and matched together
  • Wait till Notifarr supports the default condition in the groups "default": true
  • Once Notifiarr supports the default condition in the groups, review QP so they don't have the same CF as in the groups set they are allowed to use!
  • Recheck quality_profiles excludes

Requirements

@TRaSH- TRaSH- marked this pull request as draft December 15, 2024 21:10
@github-actions github-actions bot added the Area: Backend Backend Changes, not related to a specific section label Dec 15, 2024
@TRaSH- TRaSH- changed the title feat(guide-sync): Added more groups feat(guide-sync): Added more groups for the 3rd party sync applications Dec 15, 2024
@TRaSH- TRaSH- changed the title feat(guide-sync): Added more groups for the 3rd party sync applications feat(guide-sync): Add more groups for the 3rd party sync applications Dec 15, 2024
@bakerboy448
Copy link
Contributor

https://discord.com/channels/492590071455940612/1317973099966369852

@TRaSH- TRaSH- self-assigned this Dec 26, 2024
`"required": true` = It means you cant use that group without that format, it can not be turned off!
@TRaSH- TRaSH- marked this pull request as ready for review January 12, 2025 21:15
@TRaSH- TRaSH- added the Do Not Merge Do Not Merge label Jan 12, 2025
@github-actions github-actions bot added the Status: Conflicted Pull Request is Conflicted label Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Backend Backend Changes, not related to a specific section Do Not Merge Do Not Merge Status: Conflicted Pull Request is Conflicted
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants