Skip to content
This repository has been archived by the owner on Jun 15, 2021. It is now read-only.

release size estimation is off #284

Open
ghost opened this issue May 18, 2016 · 2 comments
Open

release size estimation is off #284

ghost opened this issue May 18, 2016 · 2 comments

Comments

@ghost
Copy link

ghost commented May 18, 2016

I don't know how pynab is estimating file size but it's off for me. I almost wonder if it's due to the fact that I'm running on an LZ4 compressed ZFS filesystem.

Consider the attached screenshot. My indexer is "A My PyNab" (I was hoping alpha sort affects search order)

screen shot 2016-05-18 at 16 12 01

For the exact same release NZBs.org says it's 1.9GB while PyNab says it's 1.6GB.

@gkoh
Copy link
Contributor

gkoh commented May 18, 2016

pynab's unrar module has had trouble in the past, initially I thought this was related to #280, so I had a look at the rar headers for the aforementioned release.
It looks like this (according to unrar):

 unrar va -kb 12.Monkeys.S02E04.1080p.WEB-DL.DD5.1.H.264-VietHD.part01.rar 

UNRAR 5.30 beta 6 freeware      Copyright (c) 1993-2015 Alexander Roshal

Archive: 12.Monkeys.S02E04.1080p.WEB-DL.DD5.1.H.264-VietHD.part01.rar
Details: RAR 4, volume

 Attributes      Size    Packed Ratio    Date    Time   Checksum  Name
----------- ---------  -------- ----- ---------- -----  --------  ----
    ..A.... 1761846484  24999870  -->  2016-05-11 01:17  259FE182  12.Monkeys.S02E04.1080p.WEB-DL.DD5.1.H.264-VietHD.mkv
----------- ---------  -------- ----- ---------- -----  --------  ----
           1761846484  24999870   1%  volume 1                 

On first glance the size matches neither NZBs.org or pynab. However, if we assume binary exponent sizing, (1761846484/2^30) = 1.64GiB.
This matches pynab's sizing.

Going further, if you look at the other parts of the release, you note it includes par2 data, which adds 225+MB or so, add that to the RAR files and you get around 1.9GB, matching NZBs.org.

So ... pynab provides sizing estimates of the uncompressed rar content only, looking in lib/rars.py:check_release_files() seems to confirm this.

I guess the question is what is wanted?

  1. uncompressed RAR on disk file size (pynab current behaviour)
  2. actual download size (RAR + PAR2) (newznabplus behaviour)

@ghost
Copy link
Author

ghost commented May 19, 2016

nn+ behavior is probably warranted even if it's technically wrong. Doesn't Couchpotato also consider release size in its weighting system? Then again, I see real value in the increased accuracy...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant