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

Is there a way to stop full seasons to be downloaded? #4

Closed
efelon opened this issue Jul 2, 2018 · 22 comments
Closed

Is there a way to stop full seasons to be downloaded? #4

efelon opened this issue Jul 2, 2018 · 22 comments

Comments

@efelon
Copy link

efelon commented Jul 2, 2018

Hi,

I'm on 1.0.0 from the release tags. Recently in my feeds some full season torrents show up like:

ExampleName S02 WEBRip

In this case it was a fake coming up before the full season actually was aired, stopping further downloads. Ending up setting the favorite to:

image

Another case:

Although a series favorite already states last downloaded e.g. 6x10, upcoming full seasons like S02 are downloaded regardless of the current progress of the series.

It would be really helpful to disable "full" season downloads by setting.

In general i'm very happy with the software and also successfully updated to 1.0.0 with your script.

Best regards,
Martin

@dchang0
Copy link
Owner

dchang0 commented Jul 2, 2018

Thanks for the screenshot.

You can stop torrentwatch-xa from downloading full seasons by going into the Configure > Favorites and checking Ignore Batches. A full season is a large batch, basically. The smallest batches are two episodes; some of the fansub crews release episodes in pairs. Most people do not need to download batches of any size because they expect to get one episode at a time, so checking this box usually makes no difference to them.

To reset the Favorite, clear the two Last Downloaded textboxes from the Favorite and save the Favorite, then empty the Torrents Cache. It will then pick up the next episode it sees, which might be the current episode if it is still in the feed.

@dchang0 dchang0 closed this as completed Jul 2, 2018
@efelon
Copy link
Author

efelon commented Jul 2, 2018

I didn't realize that batch could mean that, so thanks for the explanation. I'm not aware I have touched these settings since first installation. So, full of hope I went there and:

image

As you can see this option was already enabled while these incidents happened. The latest case was an episode containing "S02E12-E13", which should also be ignored if I understood what you said correctly.

@dchang0 dchang0 reopened this Jul 2, 2018
@dchang0
Copy link
Owner

dchang0 commented Jul 2, 2018

Ah yes, if it did catch that while you had Ignore Batches checked, then getting a full season is indeed a bug. I am reopening the case.

S02E12-E13 MIGHT not be detected by my engine. I don't remember ever writing a pattern match for that syntax. I'll test it and see what happens, whether it is treated as a single episode or as a batch of two episodes.

Thanks!

@dchang0
Copy link
Owner

dchang0 commented Jul 2, 2018

Wow, I guess I did write a pattern for S02E12-E13. It is detected as 2x12 through 2x13.

So this is the same bug as getting the whole season if Ignore Batches is checked. I'll have to figure out where that's coming from.

@dchang0
Copy link
Owner

dchang0 commented Jul 2, 2018

Hi--
The logic for Ignore Batch looks fine so far. Could I ask you to copy and paste the portion of the log /tmp/twxalog where it decided to download the batches?
Hopefully you have the log set to Debug level, which would explain exactly why it thought the batch was worth downloading in spite of Ignore Batches being checked.
If you don't have the log, oh well. We'll both try to trigger the problem again. Part of the problem is that we have to wait until a fake full season or a real batch comes through a feed.
Thanks very much!

@dchang0
Copy link
Owner

dchang0 commented Jul 2, 2018

So far, I am unable to reproduce the problem by adding Favorites for batches currently visible in my feed such as "HorribleSubs Megalo Box 1-13 Unofficial Batch 480p" which is detected as 1x1-1x13
If Ignore Batches is checked, it shows up as Violet (Ignored Favorite Batch) and does not download.

@efelon
Copy link
Author

efelon commented Jul 3, 2018

I'm very sorry. I don't have the log level on debug and also can not reproduce it a second time. When it happened I was busy and had no time looking into it.

If you want quicker reproduction you can look at limetorrents. They provide a RSS for custom searches even though the links within this feed don't work anymore. But for testing matches it works ok.

@dchang0
Copy link
Owner

dchang0 commented Jul 3, 2018

LimeTorrents, huh? I'll check it out. Hopefully we can reproduce this soon. Thanks!

@efelon
Copy link
Author

efelon commented Jul 6, 2018

It happened again this night. Although I haven't enable debug log in the config file yet, I post the lines of the log anyway:

2018-07-06T03:30:15+02:00 DBG: Started processing 1 feed: YourBittorrent RSS - Television
2018-07-06T03:30:15+02:00 DBG: Match found for xxxxxx.S02.WEBRip.XviD.IdeaFilm
2018-07-06T03:30:18+02:00 INF: Started: xxxxxx.S02.WEBRip.XviD.IdeaFilm in /var/download
2018-07-06T03:30:18+02:00 DBG: Removing config cache: /var/lib/torrentwatch-xa/config_cache/torrentwatch-xa-config.cache
2018-07-06T03:30:18+02:00 DBG: Successfully wrote config file: /var/lib/torrentwatch-xa/config_cache/torrentwatch-xa.config
2018-07-06T03:30:18+02:00 DBG: Updated Favorite: xxxxxx.S02.WEBRip.XviD.IdeaFilm
2018-07-06T03:30:18+02:00 INF: Exact in cache; ignoring: xxxxxx.S02.WEBRip.XviD.IdeaFilm
2018-07-06T03:30:18+02:00 INF: Processed 1 feed: YourBittorrent RSS - Television

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

Thanks. From what I can tell, it is bypassing the batch logic and downloading it directly as a single item. If the item is still in your feed, could you turn on Configure > Interface > Show Item Debug Info and then tell me what it says at the right side of the item. It should say something like Title 1_1_30_6-1 2XFULL.

@efelon
Copy link
Author

efelon commented Jul 6, 2018

Unfortunately it is not any more. I tried the limetorrents search and got not that exact name as the one causing the trigger:

"xxxxxx.S02.WEBRip.XviD.IdeaFilm" <- seams to trigger (what caused the log above)
"xxxxxx S02 WEBRip XviD IdeaFilm" <- doesn't seam to trigger (debug: 2xFULL)

What I find strange is, that some of the 2XFULL torrents are marked as "Previously Downloaded", although I cleared all the caches. And not as "Ignored Favorite Batch" or "Old Favorite".

Can you point me to how i need to enable debug output, this would save me some time to look it up. ;)
I'll provide the debug log again next time.

Thanks for your patience.

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

If you are running 1.1.0, turning on Debug Log Level is super-easy. Go to Configure > Interface > Log Level and set it to DEBUG. Also, please check the box Configure > Interface > Show Item Debug.

I am not exactly able to reproduce the problem but got close with the show "GLOW.S02.720p.ColdFilm" It detects as 2xFULL but downloaded anyway. Different from your situation, it did not update the Favorite to 2xFULL. Something weird is going on. I will need more data to debug it.

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

BTW, Previously Downloaded will occur if the Favorite itself has been updated to 2xFULL. You need to remove the FULL part and put the last episode you successfully got.

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

Hahaha! I might have found the bad logic just by reading the code line-by-line--there is a logical case where I don't check to see if Ignore Batches is turned off but it is supposed to check. Let me fix that and see if that fixes it.

@efelon
Copy link
Author

efelon commented Jul 6, 2018

I'm still on 1.0.0. so I think I find some time tonight to update to 1.1.0. and make what you suggested.

Oh, and I did reverse the favorite to 2x10. Regardless of that it shows some as "Previously Downloaded".

Good to hear about your finding. I follow your change after you checked in.

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

Hmm. There might be more than one bug if you reversed the Favorite and it says Previously Downloaded and you also emptied all caches. But for now, let's focus on fixing this one bug, then later if the Previously Downloaded bug happens, please create a new Issue for it. With debug logging and debug item info, we'll be able to pin it down quickly.

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

Patch is up. I tested it using "GLOW.S02.720p.ColdFilm" and it is blocked if Ignore Batches is turned on. Please close the ticket if it works for you. It's up to you if you want to close it right away or wait until a second successful block of a fake FULL season. Thanks again!

@dchang0
Copy link
Owner

dchang0 commented Jul 6, 2018

I just realized that Previously Downloaded does not rely on the Favorite's Last Episode. It is entirely based on whether the item is seen in the cache. Old Favorite is the one that depends on the Favorite's Last Episode.
If you emptied all the caches but Previously Downloaded items still show up, it is possible that the caches were not really emptied. Please use the command line and check the path /var/lib/torrentwatch-xa/dl_cache and see if there are files in there that start with dl_ There should be none right after emptying all caches.
Again, please put this in a separate Issue if it happens again for you. I'm just correcting my erroneous statements earlier in this thread.

@efelon
Copy link
Author

efelon commented Jul 8, 2018

I updated my installation to 1.1.0 (master) and set logging to debug.
Let's give it some time to "feed". ;)

@dchang0
Copy link
Owner

dchang0 commented Jul 8, 2018

Cool. Thanks--I'll leave this open until you confirm it works a second time.

@efelon
Copy link
Author

efelon commented Jul 30, 2018

3 weeks have past without any other incident. I also tried some manual searches to get full season hits, but couldn't see the problem again.

From my pov you can close this issue. Thanks again for your support. I would buy you a beer, if you were around. ;)

P.S.: In some rare cases it would be helpful to have "ignore batches" per favorite and not globally. But this is not a game changer at all.

@dchang0
Copy link
Owner

dchang0 commented Jul 30, 2018

Sweet! Thanks for following-up on that.

Hmm. Ignore batches per Favorite is probably do-able. Let me add that to the TODO list.

@dchang0 dchang0 closed this as completed Jul 30, 2018
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

No branches or pull requests

2 participants