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

Providers #78

Merged
merged 4 commits into from Mar 2, 2016
Merged

Providers #78

merged 4 commits into from Mar 2, 2016

Conversation

labrys
Copy link
Contributor

@labrys labrys commented Mar 2, 2016

  • Fix rarbg rate limit
  • Fix unknown qualities for tvcuk
    • appends .hdtv.x264 to the title
  • Fix WEB-DL and WEB-Rip detections
    • Optimize regex
    • Add WEB detection
    • Add DLMux detection to tagger.py using in operator instead of regex for optimization

* appends `.hdtv.x264` to the title
* Optimize regex
* Add WEB detection
* Add DLMux detection
@@ -169,7 +169,7 @@ def search(self, search_strings, age=0, ep_obj=None): # pylint: disable=too-man
torrent_size = torrent.find_all('td')[labels.index('Size')].get_text(strip=True)
size = convert_size(torrent_size, units=units) or -1

item = title, download_url, size, seeders, leechers
item = title + '.hdtv.x264', download_url, size, seeders, leechers
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we assume that for all?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per @duramato it's a safe assumption based on their labeling of releases. If it's not an HD release, it won't have 720p or 1080p in the title so it won't parse as HD. If it's BluRay or Web-dl then the HDTV will be ignored (as we test qualities from highest to lowest). If its already labeled HDTV, double detection won't cause the check to fail.

labrys added a commit that referenced this pull request Mar 2, 2016
@labrys labrys merged commit cccdb25 into develop Mar 2, 2016
@labrys labrys deleted the providers branch March 2, 2016 22:26
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.

None yet

2 participants