Synology-Forum.nl

Packages => 3rd party Packages => Auto-Sub => Topic gestart door: stappel op 12 mei 2018, 10:51:04

Titel: mogelijk bug in ProcessName()
Bericht door: stappel op 12 mei 2018, 10:51:04

Ik denk dat er een bug in de functie ProcessName() zit.
ik vond zelf correcte subs op addic7ed maar AutoSub niet. Daarom maar eens gaan debuggen.
Ik denk dat het ergens fout gaat als ProcessName() voor het eerst wordt aangeroepen. Maar dat is slechts een vermoeden.

Debug info:
Het gaat bijvoorbeeld fout bij de serie The Expanse, AutoSub gebruikt de volgende URL voor Addic7ed.
http://www.addic7ed.com/ajax_loadShow.php?show=5513&season=3&langs=|1|17|&hd=0&hi=-1
Daar zijn bij episode 5, 2 regels met subs. de eerste heeft SVA als "version" staan. dat is de release groep.
Mijn bestand is: The.Expanse.S03E05.720p.HDTV.x264-SVA.mkv
als ik extra debug informatie in de .py files stop zie ik dat:
ProcessName(SubInfo[4]) wordt aangeroepen met: SVA in de eerste run (en daarna de volgende regel met WEB-DEFLATE). Dat klopt dus.
Het resultaat in Release is echter de eerste keer:
    {'info': u'..', 'releasegrp': None, 'source': None, 'codec': None, 'rlsgrplst': [], 'quality': None, 'distro': None}
geen releasegrp gevonden dus.
in de 2e run/regel/check met WEB-DEFLATE komt er terug:
    {'info': u'.-deflate.', 'releasegrp': u'deflate', 'source': u'web', 'codec': None, 'rlsgrplst': [u'deflate'], 'quality': None, 'distro': None}
die is dus wel gevult.

dit zie bij meerder series.
http://www.addic7ed.com/ajax_loadShow.php?show=5810&season=2&langs=|1|17|&hd=0&hi=-1
eps 21, ook de eerste keer blijft de release grp op None staan.

Titel: Re: mogelijk bug in ProcessName()
Bericht door: stappel op 12 mei 2018, 11:26:14

Volgens mij komt het hierdoor:
    show_dict['info'] = show_dict['info'][Pos:]
in de file ProcessFilename.py

Als er GEEN codec en  GEEN quality en GEEN distro, etc gevonden wordt in de infon string, staat Pos nog op de lengte van de gehele string en wordt dus geheel verwijderd en wordt show_dict['info'] een lege string.

Er moet dus (vermoedelijk) een check bij komen of Pos nog de orginele Pos is en dan skippen.
Als ik even snel deze regel uit comment, dan vindt hij 5 extra subs in mijn library. allemaal correcte downloads.

Titel: Re: mogelijk bug in ProcessName()
Bericht door: Ben(V) op 12 mei 2018, 13:28:16

Klopt die bug was al gevonden en gecorrigeerd.
Oplossing komt met de volgende release mee.

Prima analyse overigens.

Titel: Re: mogelijk bug in ProcessName()
Bericht door: stappel op 12 mei 2018, 20:54:38

Ok.
er is geen development branch in github?

Titel: Re: mogelijk bug in ProcessName()
Bericht door: Ben(V) op 13 mei 2018, 00:59:18

Nee development die ik alleen lokaal.
Meestal release ik de bug fixies vrij snel,maar door omstsndigheden loop ik nu wat achter.