Skip to content

Update BLU encode min resolution#1271

Open
vmt5m3mjb wants to merge 1 commit intoAudionut:masterfrom
vmt5m3mjb:patch-4
Open

Update BLU encode min resolution#1271
vmt5m3mjb wants to merge 1 commit intoAudionut:masterfrom
vmt5m3mjb:patch-4

Conversation

@vmt5m3mjb
Copy link

@vmt5m3mjb vmt5m3mjb commented Feb 27, 2026

https://blutopia.cc/pages/18#resolution-requirements

4.2 Encodes must be at least 720p.

Summary by CodeRabbit

Release Notes

  • Enhancements
    • Added validation to prevent uploading ENCODE releases with resolutions below 720p. Uploads with 480p, 480i, 576p, or 576i resolutions will now be rejected to maintain quality standards.

@github-actions
Copy link

Thanks for taking the time to contribute to this project. Upload Assistant is currently in a complete rewrite, and no new development is being conducted on this python source at this time.

If you have come this far, please feel free to leave open, any pull requests regarding new sites being added to the source, as these can serve as the baseline for later conversion.

If your pull request relates to a critical bug, this will be addressed in this code base, and a new release published as needed.

If your pull request only addresses a quite minor bug, it is not likely to be addressed in this code base.

Details for the new code base will follow at a later date.

@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8a1ee44 and d43c693.

📒 Files selected for processing (1)
  • src/trackers/BLU.py

📝 Walkthrough

Walkthrough

A new validation guard is added to the BLU tracker that rejects ENCODE-type releases with resolutions of 576p, 576i, 480p, or 480i by printing an error and returning False (unless in unattended mode), enforcing a minimum 720p threshold for such uploads.

Changes

Cohort / File(s) Summary
ENCODE Resolution Validation
src/trackers/BLU.py
Added guard clause that rejects ENCODE releases with resolutions ≤480p/≤576p, printing an error and returning False unless unattended mode is active. Check positioned before existing valid_mi_settings validation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A guard stands tall, both fierce and true,
Blocking encodes that won't do,
Five-seven-two must say goodbye,
Seven-two-oh reaches for the sky! ✨
Quality hops forward with delight!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding a guard that enforces a minimum resolution requirement for BLU encode uploads, preventing uploads below 720p.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant